-
-
Notifications
You must be signed in to change notification settings - Fork 55
🐛 Fix Examples bugs #753
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🐛 Fix Examples bugs #753
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #753 +/- ##
=======================================
Coverage 96.65% 96.65%
=======================================
Files 15 15
Lines 778 778
Branches 76 76
=======================================
Hits 752 752
Misses 20 20
Partials 6 6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@all-contributors please add @Evergreenies for code |
I've put up a pull request to add @Evergreenies! 🎉 |
This PR is raised against the issue the I raised recently - #752
The issues has been fixed while this PR changes are pointed as below -
While I was digging into this repo
encountered some issues in examples gives to this repo. I have listed those below:
authx/examples/examples/basic_auth.py
Line 55 in 2e896a9
authx/examples/examples/refresh_token.py
Line 63 in 2e896a9
authx/examples/tests/test_examples.py
Line 265 in 2e896a9
authx/examples/examples/token_locations.py
Line 75 in 2e896a9
authx/examples/examples/token_locations.py
Line 95 in 2e896a9
unset_jwt_cookies
in authx so renamed it tounset_cookies
)authx/examples/examples/token_locations.py
Line 108 in 2e896a9
get_token_location
so removed and took location forRequestPaylod