-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
A way to make flask-caching optional / use a different caching mechanism #15271
Comments
Hi @michalc, This should have followed our responsible disclosure policy that is explicitly declared when you create a security issue on Github
We follow Apache security Guidelines has stated above, so please send a copy to [email protected] and we'll followup on this there. Thank you |
My apologies… I guess I (mis-)assumed that since the vulnerability in flask-caching is public, there was nothing remaining to disclose or report, and all discussion could be public. |
IMHO the CVE is bogus. If someone has write access to your cache's storage, you are most likely already compromised. |
did u fix @michalc ? |
Is your feature request related to a problem? Please describe.
flask-caching has a published vulnerability https://nvd.nist.gov/vuln/detail/CVE-2021-33026#vulnCurrentDescriptionTitle where access to the cache store would lead to authorised code execution in Superset. Unfortunately, a fix does not appear available at this time.
Describe the solution you'd like
A way to not use Flask caching in Superset.
Describe alternatives you've considered
Forking Superset...
Additional context
pallets-eco/flask-caching#209
apache/airflow#16541
The text was updated successfully, but these errors were encountered: