Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Async All the Way#621

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
padrepitufo wants to merge6 commits intoauth0:master
base:master
Choose a base branch
Loading
frompadrepitufo:master
Draft

Conversation

@padrepitufo
Copy link

@padrepitufopadrepitufo commentedMay 18, 2024
edited
Loading

Changes

I am probably using this wrong and need guidance.

I want to useauth0-python asynchronously. My set up is close tothis tutorial. I noticed it depends on PyJWT and I realized the library blocks. I have made anaccompanying PR (that this PR points to) but I doubt I'm the first one to use this - I'm probably just using it wrong.

Main change in usage would be (using the tutorial example as my sample):

try:signing_key=self.jwks_client.get_signing_key_from_jwt(# 👈 this lines changestoken.credentials    ).keyexceptjwt.exceptions.PyJWKClientErroraserror:raiseUnauthorizedException(str(error))exceptjwt.exceptions.DecodeErroraserror:raiseUnauthorizedException(str(error))

becomes

try:payload=awaitself.jwks_client.get_signing_key_from_jwt(# 👈 this lines changestoken.credentials    )signing_key=payload.keyexceptjwt.exceptions.PyJWKClientErroraserror:raiseUnauthorizedException(str(error))exceptjwt.exceptions.DecodeErroraserror:raiseUnauthorizedException(str(error))

References

... None

Testing

Please describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

  • [ not yet ] This change adds unit test coverage
  • [ not yet ] This change adds integration test coverage
  • [ partially ] This change has been tested on the latest version of the platform/language or why notno test coverage added because I suspect I'm doing this wrong anyways

Checklist

@padrepitufo
Copy link
Author

Any guidance that@evansims@jpadilla@adamjmcgrath could lend would be much appreciated

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@padrepitufo

[8]ページ先頭

©2009-2025 Movatter.jp