-
-
Notifications
You must be signed in to change notification settings - Fork 298
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
16.0 add fastapi auth #404
base: 16.0
Are you sure you want to change the base?
Conversation
@sebastienbeau If I'm not mistaken this work origins from https://github.com/akretion/rest-authenticable. BTW happy new year :) |
Yes it come from here. |
AFAIS 95% of the code would be in the standalone module, only a fraction goes into fastapi integration.
IMO is better to rollback to the other approach before you start spreading it 😉 In 4 years, when @lmignon will refactor these apis on the next api framework you'll thank yourself 😜 |
Thanks for your feedback, we are going to re-split the module, @paradoxxxzero will work on it. |
837996c
to
9ce4f93
Compare
add auth base on client side cookie in base module using itsdangerous add demo data and data
7392e82
to
0f695ae
Compare
…iration is updated after each call
This PR depends on #438 |
I'll fix the error into the extendable-pydantic lib |
@paradoxxxzero The compatibility issue is now fixed into extendable-pydantic... You've one failing test 😏 |
b7069de
to
bf1fd2c
Compare
bf1fd2c
to
e388aa4
Compare
12218e8
to
052e15f
Compare
Introduce a new authentication based on partner, you can use it instead of external auth provider like keycloak
This module is linked to fastapi (it can maybe more generic in futur version).