Added dependency that allows endpoints to accept in access tokens OR api keys#14
Open
amanmibra wants to merge 1 commit intoPropelAuth:mainfrom
Open
Added dependency that allows endpoints to accept in access tokens OR api keys#14amanmibra wants to merge 1 commit intoPropelAuth:mainfrom
amanmibra wants to merge 1 commit intoPropelAuth:mainfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Created
AccessTokenOrAPIKeyDependency(could use a better name)Problem: Current set of security dep wrappers only support Propel access tokens and not api key use.
Proposed Solution:
AccessTokenOrAPIKeyDependencycan be catalyst towards a more complete solution.As of right now, this is tailored to my current use case, which assumes the user is only associated with one organization and does not handle personal API keys, and then returns an "Org" object (probably ideal if it was in the
propelauth_pylibrary instead), which is the return structure of the get_org api call.I think the actual solution would just be more classes like with this one with different use cases that a user can choose. Let me know how I can help! Just wanted to kickstart this effort at the very least :)