Skip to content

Commit

Permalink
[#1784] fix runtime configuration for django-log-outgoing-requests (#75)
Browse files Browse the repository at this point in the history
* [#1784] fix runtime configuration for django-log-outgoing-requests

Previously, setting the `LOG_REQUESTS` setting to `False` caused the
runtime configuration of django-log-outgoing-requests to break. This was
caused by no handlers being configured for its logger.

As the runtime configuration at the moment only allows configuring
saving to the database (or not) the `save_outgoing_requests` should
always be configured in the `LOGGING` setting.
  • Loading branch information
SonnyBA authored Oct 4, 2024
1 parent 026f44b commit 896bd40
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion open_api_framework/conf/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -501,7 +501,7 @@
"handlers": (
["log_outgoing_requests", "save_outgoing_requests"]
if LOG_REQUESTS
else []
else ["save_outgoing_requests"]
),
"level": "DEBUG",
"propagate": True,
Expand Down Expand Up @@ -1041,3 +1041,9 @@ def init_sentry(before_send: Callable | None = None):
# CSP_SANDBOX # too much

CSP_UPGRADE_INSECURE_REQUESTS = False # TODO enable on production?


#
# Django Solo
#
SOLO_CACHE = "default"

0 comments on commit 896bd40

Please sign in to comment.