fix(openai): remove openai API key parsing entirely (#15626)

## Description
Closes #15585.

This PR removes API key parsing from the openai integration for two
reasons:
1. We were not using this anywhere (we used to tag the last 4 digits of
the API key similar to how OpenAI formats theirs in UI, but tagging this
was removed quite some time ago)
2. This caused errors in use cases involving `azure-identity` that
involved API keys as functions (i.e. not directly storing API keys in
app/memory), because we always assumed this was a string.

<!-- Provide an overview of the change and motivation for the change -->
## Testing
<!-- Describe your testing strategy or note what tests are included -->

## Risks

<!-- Note any risks associated with this change, or "None" if no risks
-->

## Additional Notes

<!-- Any other information that would be helpful for reviewers -->

Author: Yun-Kim

ddtrace/contrib/internal/openai/patch.py

@@ -185,11 +185,6 @@ def patched_client_init(openai, pin, func, instance, args, kwargs):
     func(*args, **kwargs)
     integration = openai._datadog_integration
     integration._client = instance
-    api_key = kwargs.get("api_key")
-    if api_key is None:
-        api_key = instance.api_key
-    if api_key is not None:
-        integration.user_api_key = api_key
     return
 
 

ddtrace/llmobs/_integrations/openai.py

@@ -50,20 +50,7 @@ def __init__(self, integration_config, openai):
         # object that is strongly linked with configuration.
         super().__init__(integration_config)
         self._openai = openai
-        self._user_api_key = None
         self._client = None
-        if self._openai.api_key is not None:
-            self.user_api_key = self._openai.api_key
-
-    @property
-    def user_api_key(self) -> Optional[str]:
-        """Get a representation of the user API key for tagging."""
-        return self._user_api_key
-
-    @user_api_key.setter
-    def user_api_key(self, value: str) -> None:
-        # Match the API key representation that OpenAI uses in their UI.
-        self._user_api_key = "sk-...%s" % value[-4:]
 
     def trace(self, pin: Pin, operation_id: str, submit_to_llmobs: bool = False, **kwargs: Dict[str, Any]) -> Span:
         traced_operations = (

releasenotes/notes/fix-openai-remove-api-key-c372e143ca7f00f5.yaml

@@ -0,0 +1,4 @@
+---
+fixes:
+  - |
+    openai: Resolves an issue where instantiating an OpenAI client with a non-string API key resulted in parsing issues.