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

one canonical Flask debug config #4683

Merged
merged 2 commits into from
Nov 2, 2023

Conversation

jmcrawford45
Copy link
Contributor

Flask configuration variables are case sensitive. Therefore, DEBUG and debug would be two different configuration variables, if both were used.

However, by convention and per Flask's documentation, the correct configuration variable for activating debug mode is DEBUG, not debug.

In other words, Flask recognizes DEBUG as the configuration variable to control the activation of debug mode. Using debug would not have the desired effect unless you've added custom code in your application to also check for a debug configuration variable.

@coveralls
Copy link

coveralls commented Nov 2, 2023

Coverage Status

coverage: 75.275%. first build when pulling e02a415 on jmcrawford45:SECGUIDE-4012 into d263fe5 on Netflix:main.

@jmcrawford45 jmcrawford45 merged commit c90d6b2 into Netflix:main Nov 2, 2023
22 checks passed
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.

3 participants