-
-
Notifications
You must be signed in to change notification settings - Fork 279
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
add auth base on client side cookie in base module using itsdangerous add demo data and data
- add view, wizard (to send reminder in migration case) - add end2end test - fix cookies and api
@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. |
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).