Add uri decode for OTEL_RESOUCE_ATTRIBUTES

Author: shalevr

Diff for: CHANGELOG.md

@@ -7,6 +7,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+- Add uri decode values from OTEL_RESOURCE_ATTRIBUTES
+  ([#3046](https://github.com/open-telemetry/opentelemetry-python/pull/3046))
 - Add missing entry points for OTLP/HTTP exporter
   ([#3027](https://github.com/open-telemetry/opentelemetry-python/pull/3027))
 

Diff for: opentelemetry-sdk/src/opentelemetry/sdk/resources/__init__.py

@@ -61,6 +61,7 @@
 import os
 import sys
 import typing
+from urllib import parse
 from json import dumps
 
 import pkg_resources
@@ -289,7 +290,8 @@ def detect(self) -> "Resource":
                         exc,
                     )
                     continue
-                env_resource_map[key.strip()] = value.strip()
+                value_uri_decoded = parse.unquote(value.strip())
+                env_resource_map[key.strip()] = value_uri_decoded
 
         service_name = os.environ.get(OTEL_SERVICE_NAME)
         if service_name:

Diff for: opentelemetry-sdk/tests/resources/test_resources.py

@@ -492,6 +492,13 @@ def test_invalid_key_value_pairs(self):
             resources.Resource({"k": "v", "k2": "v2", "foo": "bar=baz"}),
         )
 
+    def test_multiple_with_uri_decode(self):
+        detector = resources.OTELResourceDetector()
+        os.environ[resources.OTEL_RESOURCE_ATTRIBUTES] = "key=value%20test%0A, key2=value+%202"
+        self.assertEqual(
+            detector.detect(), resources.Resource({"key": "value test\n", "key2": "value+ 2"})
+        )
+
     @mock.patch.dict(
         os.environ,
         {resources.OTEL_SERVICE_NAME: "test-srv-name"},