Skip to content

Commit

Permalink
🐛(logging) fix wrong settings for tracking logs
Browse files Browse the repository at this point in the history
We were initializing TRACKING_IGNORE_URL_PATTERNS to None instead of a list
of url to ignore when writing tracking logs, making each requests track a
TypeError.
  • Loading branch information
rmoch committed Jan 31, 2019
1 parent 7a40df0 commit bfcaa3c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion config/cms/docker_run_production.py
Original file line number Diff line number Diff line change
Expand Up @@ -339,7 +339,7 @@

# Event Tracking
TRACKING_IGNORE_URL_PATTERNS = config(
"TRACKING_IGNORE_URL_PATTERNS", default=None, formatter=json.loads
"TRACKING_IGNORE_URL_PATTERNS", default=TRACKING_IGNORE_URL_PATTERNS, formatter=json.loads
)

# Heartbeat
Expand Down
2 changes: 1 addition & 1 deletion config/lms/docker_run_production.py
Original file line number Diff line number Diff line change
Expand Up @@ -471,7 +471,7 @@

# Event Tracking
TRACKING_IGNORE_URL_PATTERNS = config(
"TRACKING_IGNORE_URL_PATTERNS", default=None, formatter=json.loads
"TRACKING_IGNORE_URL_PATTERNS", default=TRACKING_IGNORE_URL_PATTERNS, formatter=json.loads
)

# SSL external authentication settings
Expand Down

0 comments on commit bfcaa3c

Please sign in to comment.