Fixed ruff

Author: rads-1996

CHANGELOG.md

@@ -10,8 +10,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 > Use [this search for a list of all CHANGELOG.md files in this repo](https://github.com/search?q=repo%3Aopen-telemetry%2Fopentelemetry-python-contrib+path%3A**%2FCHANGELOG.md&type=code).
 
 ## Unreleased
-- `opentelemetry-util-http` Added support for redacting specific url query string values and url credentials in instrumentations
-  ([#3508](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3508))
 
 ## Version 1.34.0/0.55b0 (2025-06-04)
 
@@ -24,6 +22,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - `opentelemetry-instrumentation-aiokafka` Add instrumentation of `consumer.getmany` (batch)
   ([#3257](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3257))
+- `opentelemetry-util-http` Added support for redacting specific url query string values and url credentials in instrumentations
+  ([#3508](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3508))
 
 ### Fixed
 

instrumentation/opentelemetry-instrumentation-httpx/tests/test_httpx_integration.py

@@ -1316,8 +1316,11 @@ def test_remove_sensitive_params(self):
             )
         else:
             # If credentials are removed completely, the query string should still be redacted
-            self.assertIn("http://mock/status/200?sig=REDACTED", actual_url,
-                        f"Basic URL structure is incorrect: {actual_url}")
+            self.assertIn(
+                "http://mock/status/200?sig=REDACTED",
+                actual_url,
+                f"Basic URL structure is incorrect: {actual_url}",
+            )
 
 
 class TestAsyncIntegration(BaseTestCases.BaseManualTest):
@@ -1393,12 +1396,15 @@ def test_remove_sensitive_params(self):
 
         if "@" in actual_url:
             self.assertEqual(
-            span.attributes[SpanAttributes.HTTP_URL],
-            "http://REDACTED:REDACTED@mock/status/200?Signature=REDACTED",
+                span.attributes[SpanAttributes.HTTP_URL],
+                "http://REDACTED:REDACTED@mock/status/200?Signature=REDACTED",
             )
         else:
-            self.assertIn("http://mock/status/200?Signature=REDACTED", actual_url,
-                     f"If credentials are removed, the query string still should be redacted {actual_url}")
+            self.assertIn(
+                "http://mock/status/200?Signature=REDACTED",
+                actual_url,
+                f"If credentials are removed, the query string still should be redacted {actual_url}",
+            )
 
 
 class TestSyncInstrumentationIntegration(BaseTestCases.BaseInstrumentorTest):

util/opentelemetry-util-http/tests/test_redact_query_parameters.py

@@ -70,16 +70,16 @@ def test_password_key_not_in_redact_list(self):
         )
 
     def test_url_with_at_symbol_in_path_and_query(self):
-        url = "https://github.com/p@th?foo=b@r"
+        url = "https://example.com/p@th?foo=b@r"
         self.assertEqual(
-            redact_query_parameters(url), "https://github.com/p@th?foo=b@r"
+            redact_query_parameters(url), "https://example.com/p@th?foo=b@r"
         )
 
     def test_aws_access_key_with_real_format(self):
-        url = "https://microsoft.com?AWSAccessKeyId=AKIAIOSFODNN7"
+        url = "https://mock.com?AWSAccessKeyId=AKIAIOSFODNN7"
         self.assertEqual(
             redact_query_parameters(url),
-            "https://microsoft.com?AWSAccessKeyId=REDACTED",
+            "https://mock.com?AWSAccessKeyId=REDACTED",
         )
 
     def test_signature_parameter(self):

util/opentelemetry-util-http/tests/test_remove_credentials.py

@@ -19,16 +19,16 @@
 
 class TestRemoveUrlCredentials(unittest.TestCase):
     def test_remove_no_credentials(self):
-        url = "http://opentelemetry.io:8080/test/path?query=value"
+        url = "http://mock/status/200/test/path?query=value"
         cleaned_url = remove_url_credentials(url)
         self.assertEqual(cleaned_url, url)
 
     def test_remove_credentials(self):
-        url = "http://someuser:somepass@opentelemetry.io:8080/test/path?sig=value"
+        url = "http://someuser:somepass@mock/status/200/test/path?sig=value"
         cleaned_url = remove_url_credentials(url)
         self.assertEqual(
             cleaned_url,
-            "http://REDACTED:REDACTED@opentelemetry.io:8080/test/path?sig=value",
+            "http://REDACTED:REDACTED@mock/status/200/test/path?sig=value",
         )
 
     def test_remove_credentials_ipv4_literal(self):