Addressed comments

Author: rads-1996

CHANGELOG.md

@@ -17,6 +17,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - `opentelemetry-instrumentation-flask`: Do not record `http.server.duration` metrics for excluded URLs.
   ([#3794](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3794))
+- `opentelemetry-instrumentation-django`: Fixes invalid type at WSGI request headers and attributes collection.
+  ([#3731](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3731))
 - `opentelemetry-instrumentation-botocore`: migrate off the deprecated events API to use the logs API
   ([#3624](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3624))
 - `opentelemetry-instrumentation-dbapi`: fix crash retrieving libpq version when enabling commenter with psycopg
@@ -57,8 +59,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   ([#3673](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3673))
 - `opentelemetry-instrumentation-starlette`/`opentelemetry-instrumentation-fastapi`: Fixes a crash when host-based routing is used
   ([#3507](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3507))
-- `opentelemetry-instrumentation-django`: Fixes invalid type at WSGI request headers and attributes collection.
-  ([#3731](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3731))
 - Fix documentation order of sections and headers for Django, Flask, MySQL, mysqlclient, psycopg, psycopg2, pymysql, sqlalchemy instrumentations.
   ([#3719](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3719))
 - `opentelemetry-instrumentation-asgi` Fixed an issue where FastAPI reports IP instead of URL.

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

@@ -180,14 +180,7 @@ def format_request_objects_in_headers(attributes):
         for _, value_list in attributes.items():
             for index, value in enumerate(value_list):
                 if isinstance(value, HttpRequest):
-                    try:
-                        method = getattr(value, "method", "UNKNOWN")
-                        request_path = getattr(value, "path", "UNKNOWN")
-                        value_list[index] = (
-                            f"HttpRequest({method} {request_path})"
-                        )
-                    except Exception:  # pylint: disable=broad-exception-caught
-                        value_list[index] = "HttpRequest(...)"
+                    value_list[index] = str(value)
         return attributes
 
     @staticmethod

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

@@ -1031,7 +1031,7 @@ def test_wsgi_request_in_header_is_properly_formatted(self):
         }
         expected_attributes = {
             "http.request.header.test_wsgirequest_header": [
-                "HttpRequest(GET /test/path)"
+                str(mock_wsgi_request)
             ]
         }
 
@@ -1055,7 +1055,7 @@ def test_wsgi_request_handles_extraction_error(self):
             "http.request.header.test_wsgirequest_header": [mock_wsgi_request]
         }
         expected_attributes = {
-            "http.request.header.test_wsgirequest_header": ["HttpRequest(...)"]
+            "http.request.header.test_wsgirequest_header": str(mock_wsgi_request)
         }
 
         formatted_attributes = (
@@ -1077,7 +1077,7 @@ def test_handles_http_request_as_well(self):
         }
         expected_attributes = {
             "http.request.header.test_httprequest_header": [
-                "HttpRequest(POST /api/data)"
+                str(mock_http_request)
             ]
         }
 
@@ -1101,7 +1101,7 @@ def test_regular_header_values_are_preserved(self):
         }
         expected_attributes = {
             "http.request.header.test_wsgirequest_header": [
-                "HttpRequest(GET /test/path)"
+                str(mock_wsgi_request)
             ],
             "http.request.header.test_regular_header": ["regular-value"],
         }