Skip to content

Commit 015163c

Browse files
piotrjeztammy-baylis-swixrmx
authored
opentelemetry-instrumentation-django: improve docs for response_hook with examples of providing attributes from middlewares (#3923)
* instrumentation/django: improve docs for response hooks * changelog * modify new sections as subsections under "Request and Response hooks" --------- Co-authored-by: Tammy Baylis <96076570+tammy-baylis-swi@users.noreply.github.com> Co-authored-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
1 parent 337a986 commit 015163c

File tree

2 files changed

+75
-5
lines changed
  • instrumentation/opentelemetry-instrumentation-django/src/opentelemetry/instrumentation/django

2 files changed

+75
-5
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
2828
- `opentelemetry-instrumentation-aiohttp-server`: add support for custom header captures via `OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SERVER_REQUEST` and `OTEL_INSTRUMENTATION_HTTP_CAPTURE_HEADERS_SERVER_RESPONSE`
2929
([#3916](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3916))
3030
- `opentelemetry-instrumentation-redis`: add support for `suppress_instrumentation` context manager for both sync and async Redis clients and pipelines
31+
- `opentelemetry-instrumentation-django`: improve docs for response_hook with examples of providing attributes from middlewares
32+
([#3923](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3923))
3133
- Update for Log SDK breaking changes. Rename InMemoryLogExporter to InMemoryLogRecordExporter in several tests
3234
([#3850](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3589))
3335

instrumentation/opentelemetry-instrumentation-django/src/opentelemetry/instrumentation/django/__init__.py

Lines changed: 73 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
will exclude requests such as ``https://site/client/123/info`` and ``https://site/xyz/healthcheck``.
4646
4747
Request attributes
48-
********************
48+
******************
4949
To extract attributes from Django's request object and use them as span attributes, set the environment variable
5050
``OTEL_PYTHON_DJANGO_TRACED_REQUEST_ATTRS`` to a comma delimited list of request attribute names.
5151
@@ -57,10 +57,10 @@
5757
5858
will extract the ``path_info`` and ``content_type`` attributes from every traced request and add them as span attributes.
5959
60-
Django Request object reference: https://docs.djangoproject.com/en/3.1/ref/request-response/#attributes
60+
* `Django Request object reference <https://docs.djangoproject.com/en/5.2/ref/request-response/#attributes>`_
6161
6262
Request and Response hooks
63-
***************************
63+
**************************
6464
This instrumentation supports request and response hooks. These are functions that get called
6565
right after a span is created for a request and right before the span is finished for the response.
6666
The hooks can be configured as follows:
@@ -77,8 +77,76 @@ def response_hook(span, request, response):
7777
7878
DjangoInstrumentor().instrument(request_hook=request_hook, response_hook=response_hook)
7979
80-
Django Request object: https://docs.djangoproject.com/en/3.1/ref/request-response/#httprequest-objects
81-
Django Response object: https://docs.djangoproject.com/en/3.1/ref/request-response/#httpresponse-objects
80+
* `Django Request object <https://docs.djangoproject.com/en/5.2/ref/request-response/#httprequest-objects>`_
81+
* `Django Response object <https://docs.djangoproject.com/en/5.2/ref/request-response/#httpresponse-objects>`_
82+
83+
Adding attributes from middleware context
84+
#########################################
85+
In many Django applications, certain request attributes become available only *after*
86+
specific middlewares have executed. For example:
87+
88+
- ``django.contrib.auth.middleware.AuthenticationMiddleware`` populates ``request.user``
89+
- ``django.contrib.sites.middleware.CurrentSiteMiddleware`` populates ``request.site``
90+
91+
Because the OpenTelemetry instrumentation creates the span **before** Django middlewares run,
92+
these attributes are **not yet available** in the ``request_hook`` stage.
93+
94+
Therefore, such attributes should be safely attached in the **response_hook**, which executes
95+
after Django finishes processing the request (and after all middlewares have completed).
96+
97+
Example: Attaching the authenticated user and current site to the span:
98+
99+
.. code:: python
100+
101+
def response_hook(span, request, response):
102+
# Attach user information if available
103+
if request.user.is_authenticated:
104+
span.set_attribute("enduser.id", request.user.pk)
105+
span.set_attribute("enduser.username", request.user.get_username())
106+
107+
# Attach current site (if provided by CurrentSiteMiddleware)
108+
if hasattr(request, "site"):
109+
span.set_attribute("site.id", getattr(request.site, "pk", None))
110+
span.set_attribute("site.domain", getattr(request.site, "domain", None))
111+
112+
DjangoInstrumentor().instrument(response_hook=response_hook)
113+
114+
This ensures that middleware-dependent context (like user or site information) is properly
115+
recorded once Django’s middleware stack has finished execution.
116+
117+
Custom Django middleware can also attach arbitrary data to the ``request`` object,
118+
which can later be included as span attributes in the ``response_hook``.
119+
120+
* `Django middleware reference <https://docs.djangoproject.com/en/5.2/topics/http/middleware/>`_
121+
122+
Best practices
123+
##############
124+
- Use **response_hook** (not request_hook) when accessing attributes added by Django middlewares.
125+
- Common middleware-provided attributes include:
126+
127+
- ``request.user`` (AuthenticationMiddleware)
128+
- ``request.site`` (CurrentSiteMiddleware)
129+
130+
- Avoid adding large or sensitive data (e.g., passwords, session tokens, PII) to spans.
131+
- Use **namespaced attribute keys**, e.g., ``enduser.*``, ``site.*``, or ``custom.*``, for clarity.
132+
- Hooks should execute quickly — avoid blocking or long-running operations.
133+
- Hooks can be safely combined with OpenTelemetry **Context propagation** or **Baggage**
134+
for consistent tracing across services.
135+
136+
* `OpenTelemetry semantic conventions <https://opentelemetry.io/docs/specs/semconv/http/http-spans/>`_
137+
138+
Middleware execution order
139+
##########################
140+
In Django’s request lifecycle, the OpenTelemetry `request_hook` is executed before
141+
the first middleware runs. Therefore:
142+
143+
- At `request_hook` time → only the bare `HttpRequest` object is available.
144+
- After middlewares → `request.user`, `request.site` etc. become available.
145+
- At `response_hook` time → all middlewares (including authentication and site middlewares)
146+
have already run, making it the correct place to attach these attributes.
147+
148+
Developers who need to trace attributes from middlewares should always use `response_hook`
149+
to ensure complete and accurate span data.
82150
83151
Capture HTTP request and response headers
84152
*****************************************

0 commit comments

Comments
 (0)