feat: address comments

Author: SoulPancake

openfga_sdk/credentials.py

@@ -218,7 +218,7 @@ def validate_credentials_config(self):
                 or none_or_empty(self.configuration.api_issuer)
             ):
                 raise ApiValueError(
-                    "configuration `{}` requires client_id, client_secret and api_issuer defined for client_credentials method."
+                    f"configuration `{self.configuration}` requires client_id, client_secret and api_issuer defined for client_credentials method."
                 )
 
             # Normalize blank/whitespace values to None
@@ -235,7 +235,7 @@ def validate_credentials_config(self):
                 self.configuration.scopes = None
             if isinstance(self.configuration.scopes, list):
                 self.configuration.scopes = [
-                    s
+                    s.strip()
                     for s in self.configuration.scopes
                     if isinstance(s, str) and s.strip()
                 ]

openfga_sdk/oauth2.py

@@ -76,7 +76,7 @@ async def _obtain_token(self, client):
         # Add scope parameter if scopes are configured
         if configuration.scopes is not None:
             if isinstance(configuration.scopes, list):
-                scope_str = " ".join(s for s in configuration.scopes if s and s.strip())
+                scope_str = " ".join(s.strip() for s in configuration.scopes if s and s.strip())
             else:
                 scope_str = (
                     configuration.scopes.strip()

openfga_sdk/sync/oauth2.py

@@ -76,7 +76,7 @@ def _obtain_token(self, client):
         # Add scope parameter if scopes are configured
         if configuration.scopes is not None:
             if isinstance(configuration.scopes, list):
-                scope_str = " ".join(s for s in configuration.scopes if s and s.strip())
+                scope_str = " ".join(s.strip() for s in configuration.scopes if s and s.strip())
             else:
                 scope_str = (
                     configuration.scopes.strip()

test/sync/oauth2_test.py

@@ -478,4 +478,3 @@ def test_get_authentication_with_scopes_no_audience(self, mock_request):
         )
         rest_client.close()
 
-        rest_client.close()