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

Partiallly Revert "Remove workaround from check_csrf() (#6919)" #7327

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eradman
Copy link
Collaborator

@eradman eradman commented Feb 14, 2025

Tested with MULTI_ORG enabled.

What type of PR is this?

  • Bug Fix

Description

This workaround was missing if view is not None as found in https://github.com/pallets-eco/flask-wtf/pull/419/files

How is this tested?

  • Manually

With the following added to .env

REDASH_MULTI_ORG=true
REDASH_ENFORCE_CSRF=true
REDASH_ENFORCE_CSRF_FOR_AUTH_USERS=true

@eradman eradman mentioned this pull request Feb 14, 2025
5 tasks
This workaround was missing 'if view is not None ' as found in
https://github.com/pallets-eco/flask-wtf/pull/419/files

Tested with MULTI_ORG enabled.
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

Successfully merging this pull request may close these issues.

1 participant