Skip to content

Commit

Permalink
Merge pull request #4683 from jmcrawford45/SECGUIDE-4012
Browse files Browse the repository at this point in the history
one canonical Flask debug config
  • Loading branch information
jmcrawford45 authored Nov 2, 2023
2 parents d263fe5 + e02a415 commit c90d6b2
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ Changelog

Unreleased
~~~~~~~~~~~~~~~~~~~~
To avoid confusion, the debug app configuration property has been replaced with the standard DEBUG flask app config.
Added ability for new versions of LEMUR_TOKEN_SECRET via the LEMUR_TOKEN_SECRETS config option. This allows for
migration and rotation of the secret.

Expand Down
2 changes: 1 addition & 1 deletion docker/src/lemur.conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
LOG_JSON = True

CORS = os.environ.get("CORS") == "True"
debug = os.environ.get("DEBUG") == "True"
DEBUG = os.environ.get("DEBUG") == "True"


def get_random_secret(length):
Expand Down
2 changes: 1 addition & 1 deletion lemur/manage.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ def cli(script_info, config):
# These will need to be set to `True` if you are developing locally
CORS = False
debug = False
DEBUG = False
# this is the secret key used by flask session management
SECRET_KEY = "{flask_secret_key}"
Expand Down
2 changes: 1 addition & 1 deletion lemur/tests/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ def get_random_secret(length):

# These will need to be set to `True` if you are developing locally
CORS = False
debug = False
DEBUG = False

TESTING = True

Expand Down

0 comments on commit c90d6b2

Please sign in to comment.