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

Problem in Apache Superset: PyJWT 2.10 changed semantics, Flask-JWT-Extended 4.7.1 needs a configuration change #2287

Open
amotl opened this issue Nov 22, 2024 · 2 comments

Comments

@amotl
Copy link

amotl commented Nov 22, 2024

Problem

When setting up a fresh sandbox environment, PyJWT 2.10 gets installed, released on Nov 17, i.e. five days ago. That breaks a little integration test suite against Apache Superset we are running.

The initial discovery was reported here:

The detailed report and investigation happened here:

Solution

@jlucier and @vimalloc suggested at apache/superset#30995 (comment):

Have you configured flask-jwt-extended to not verify the sub claim? 'JWT_VERIFY_SUB=False'.

That probably needs to take place after upgrading to Flask-JWT-Extended 4.7.1, which includes relevant code updates?

@amotl
Copy link
Author

amotl commented Nov 22, 2024

We have been able to resolve this by updating to Flask-JWT-Extended 4.7.1 and configuring it using JWT_VERIFY_SUB = False.

@michael-s-molina
Copy link

@dpgaspar This change will break API use cases in Superset. Will this be handled by Flask AppBuilder or do we need to add a default configuration in Superset?

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