Skip to content

Python: Implement check for flask debug mode.#528

Merged
markshannon merged 5 commits into
github:rc/1.19from
tausbn:python-flask-debug
Nov 27, 2018
Merged

Python: Implement check for flask debug mode.#528
markshannon merged 5 commits into
github:rc/1.19from
tausbn:python-flask-debug

Conversation

@taus-semmle

Copy link
Copy Markdown
Contributor

Does not include a change in the change notes. This will be added in a separate PR.

Also changes the .expected file for CWE-079/ReflectedXss.ql slightly, due to a small change in the flask library stubs.

@ghost

ghost commented Nov 22, 2018

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@taus-semmle taus-semmle requested review from a team as code owners November 27, 2018 14:21
@taus-semmle taus-semmle changed the base branch from next to rc/1.19 November 27, 2018 14:21
@taus-semmle

Copy link
Copy Markdown
Contributor Author

Updated the test results and added a change note.

@hvitved hvitved removed the request for review from a team November 27, 2018 14:31
@markshannon markshannon removed request for a team November 27, 2018 15:03
* @description Running a Flask app in debug mode may allow an attacker to run arbitrary code through the Werkzeug debugger.
* @kind problem
* @problem.severity error
* @precision medium

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems low. I think this merits "high".

@markshannon markshannon merged commit 31ac33e into github:rc/1.19 Nov 27, 2018
@tausbn tausbn deleted the python-flask-debug branch February 12, 2021 18:06
cklin pushed a commit that referenced this pull request May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants