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

Missing support of :param require_token: for decorator accept_token makes this lib backward incompatible #60

Open
yurnov opened this issue Dec 13, 2023 · 2 comments

Comments

@yurnov
Copy link

yurnov commented Dec 13, 2023

Recently I found that flask-oidc was forked and back to a maintainable state, but I found that direct upgrade of flask-oidc from pypi brake some backward compatibility when decorator @oidc.accept_token is used.

In the my code I have the following:

@oidc.accept_token(require_token=True)

everywhere where authorization is required, that doesn't work in 2.x version

@dannyp11
Copy link

dannyp11 commented Mar 28, 2024

I used the optional instead

@oidc.accept_token(optional=False)

@yurnov
Copy link
Author

yurnov commented Mar 28, 2024

Hi @dannyp11,

in my case, I finally decided to replace flask-oidc with authlib

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

No branches or pull requests

2 participants