Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #132 +/- ##
==========================================
- Coverage 97.60% 97.51% -0.09%
==========================================
Files 15 15
Lines 709 724 +15
==========================================
+ Hits 692 706 +14
- Misses 17 18 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
As per our last conversation on the call, we should limit the amount of account created. For security reasons/preventing people to abuse this, I think it might be better if we do a whitelist process where users have to request to register for access. Later on if it works out, we can switch over to the model where any user can create one, but only one account. What are your thoughts on this? |
Summary of convo todayWe should prioritize security with features like these. Next steps are looking into using scopes to limit access, opening up feature to Penn Course Review (and maybe also Course Plan?), and add more products as we go. Three options discussed for limiting access:
Additional points related to PR:
|
This PR aims to add a developer page to https://platform.pennlabs.org/ where users can register client credentials for applications to access API resources on their behalf. This is useful for student developers who want to access authenticated routes on Penn Labs APIs, e.g. Penn Course Review routes.