Skip to content
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

Update authentication solution with OpenID #5

Open
NewJerseyStyle opened this issue May 7, 2024 · 4 comments
Open

Update authentication solution with OpenID #5

NewJerseyStyle opened this issue May 7, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@NewJerseyStyle
Copy link
Owner

The existing solution is not good enough and not secure

Review and consider OpenID solution with OIDC server.
Check out A Python OpenID Connect implementation
the OIC library

@NewJerseyStyle NewJerseyStyle added the enhancement New feature or request label Jun 28, 2024
@NewJerseyStyle
Copy link
Owner Author

This client also good
https://docs.authlib.org/en/latest/client/starlette.html

@NewJerseyStyle
Copy link
Owner Author

But we need to double check the use case scenarios of login/auth

ordinary users are not required to login for comment/vote

@NewJerseyStyle
Copy link
Owner Author

NewJerseyStyle commented Feb 21, 2025

Decision: Use Casbin

integrate using fastapi-authz
merge policies of modules using rbac_policy.csv

@NewJerseyStyle
Copy link
Owner Author

Can later make a middleware module of https://docs.fief.dev/getting-started/introduction/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant