Skip to content

Commit 226a001

Browse files
committed
fix django middleware
1 parent 4936180 commit 226a001

File tree

2 files changed

+10
-11
lines changed

2 files changed

+10
-11
lines changed

instrumentation/opentelemetry-instrumentation-django/src/opentelemetry/instrumentation/django/middleware/otel_middleware.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@
5353
get_excluded_urls,
5454
get_traced_request_attrs,
5555
normalise_request_header_name,
56+
normalise_response_header_name,
5657
)
5758

5859
try:
@@ -111,7 +112,6 @@ def __call__(self, request):
111112
set_status_code = None
112113
_is_asgi_supported = False
113114

114-
115115
_logger = getLogger(__name__)
116116
_attributes_by_preference = [
117117
[
@@ -361,7 +361,7 @@ def process_response(self, request, response):
361361
get_custom_headers(
362362
OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SERVER_RESPONSE
363363
),
364-
normalise_request_header_name,
364+
normalise_response_header_name,
365365
)
366366
)
367367
for key, value in custom_res_attributes.items():

instrumentation/opentelemetry-instrumentation-django/tests/test_middleware_asgi.py

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -436,7 +436,14 @@ async def test_no_op_tracer_provider(self):
436436
self.assertEqual(len(spans), 0)
437437

438438

439-
439+
@patch.dict(
440+
"os.environ",
441+
{
442+
OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SANITIZE_FIELDS: ".*my-secret.*",
443+
OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SERVER_REQUEST: "Custom-Test-Header-1,Custom-Test-Header-2,Custom-Test-Header-3,Regex-Test-Header-.*,Regex-Invalid-Test-Header-.*,.*my-secret.*",
444+
OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SERVER_RESPONSE: "Custom-Test-Header-1,Custom-Test-Header-2,Custom-Test-Header-3,my-custom-regex-header-.*,invalid-regex-header-.*,.*my-secret.*",
445+
},
446+
)
440447
class TestMiddlewareAsgiWithCustomHeaders(SimpleTestCase, TestBase):
441448
@classmethod
442449
def setUpClass(cls):
@@ -461,14 +468,6 @@ def tearDownClass(cls):
461468
super().tearDownClass()
462469
conf.settings = conf.LazySettings()
463470

464-
@patch.dict(
465-
"os.environ",
466-
{
467-
OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SANITIZE_FIELDS: ".*my-secret.*",
468-
OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SERVER_REQUEST: "Custom-Test-Header-1,Custom-Test-Header-2,Custom-Test-Header-3,Regex-Test-Header-.*,Regex-Invalid-Test-Header-.*,.*my-secret.*",
469-
OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SERVER_RESPONSE: "Custom-Test-Header-1,Custom-Test-Header-2,Custom-Test-Header-3,my-custom-regex-header-.*,invalid-regex-header-.*,.*my-secret.*",
470-
},
471-
)
472471
async def test_http_custom_request_headers_in_span_attributes(self):
473472
expected = {
474473
"http.request.header.custom_test_header_1": (

0 commit comments

Comments
 (0)