fedora-infra / flask-oidc

OpenID Connect support for Flask
24 stars 14 forks source link

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

Open yurnov opened 9 months ago

yurnov commented 9 months ago

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 commented 6 months ago

I used the optional instead

@oidc.accept_token(optional=False)
yurnov commented 6 months ago

Hi @dannyp11,

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