feat: support custom OIDC token endpoint paths

Author: wizzfi1

CHANGELOG.md

@@ -3,6 +3,14 @@
 
 ## [Unreleased](https://github.com/openfga/js-sdk/compare/v0.9.0...HEAD)
 
+### Added
+- Support for custom OIDC token endpoint paths (#141, #238)
+- Compatibility with Zitadel, Entra ID, and other OIDC providers using non-standard token paths
+
+### Fixed
+- OIDC token URL construction to handle custom paths consistently with other SDKs
+- Telemetry configuration issues in credentials flow
+
 ## v0.9.0
 
 ### [v0.9.0](https://github.com/openfga/js-sdk/compare/v0.8.1...v0.9.0) (2025-06-04)

README.md

@@ -144,6 +144,212 @@ const fgaClient = new OpenFgaClient({
 });
 ```
 
+### OIDC Token Endpoint Configuration
+
+The SDK supports custom OIDC token endpoints for compatibility with various OIDC providers.
+
+#### Default Behavior
+- When `apiTokenIssuer` is just a domain (e.g., `"auth.example.com"`), the SDK appends `/oauth/token`
+- Example: `auth.example.com` → `https://auth.example.com/oauth/token`
+
+#### Custom Token Paths
+- When `apiTokenIssuer` includes a path, that path is used as-is
+- Examples:
+  - `auth.example.com/oauth/v2` → `https://auth.example.com/oauth/v2`
+  - `https://auth.example.com/oauth/v2/token` → `https://auth.example.com/oauth/v2/token`
+
+#### Provider Examples
+
+**Zitadel:**
+```javascript
+apiTokenIssuer: 'https://auth.zitadel.example/oauth/v2/token'
+```
+
+Entra ID (Azure AD):
+```
+apiTokenIssuer: 'https://login.microsoftonline.com/tenant-id/oauth2/v2.0/token'
+```
+
+Auth0:
+
+```
+apiTokenIssuer: 'https://your-domain.auth0.com/oauth/token'
+```
+
+This ensures compatibility with OIDC providers that use non-standard token endpoint paths.
+
+```
+## 3. Create the OIDC Test File
+
+Create `tests/credentials-oidc.test.ts`:
+
+```typescript
+import nock from 'nock';
+import { OpenFgaClient, UserClientConfigurationParams } from '../src';
+import { CredentialsMethod } from '../src/credentials';
+import { baseConfig } from './helpers/default-config';
+
+describe('OIDC Token Path Handling', () => {
+  beforeEach(() => {
+    nock.disableNetConnect();
+  });
+
+  afterEach(() => {
+    nock.cleanAll();
+    nock.enableNetConnect();
+  });
+
+  const testOidcConfig = (apiTokenIssuer: string, expectedTokenUrl: string) => {
+    const config: UserClientConfigurationParams = {
+      ...baseConfig,
+      credentials: {
+        method: CredentialsMethod.ClientCredentials,
+        config: {
+          clientId: 'test-client',
+          clientSecret: 'test-secret',
+          apiTokenIssuer,
+          apiAudience: 'https://api.fga.example'
+        }
+      }
+    };
+
+    // Mock the token endpoint
+    const tokenScope = nock(expectedTokenUrl.split('/').slice(0, 3).join('/'))
+      .post(expectedTokenUrl.split('/').slice(3).join('/'))
+      .reply(200, {
+        access_token: 'test-token',
+        expires_in: 300
+      });
+
+    // Mock the FGA API call
+    const apiScope = nock('https://api.fga.example')
+      .post(`/stores/${baseConfig.storeId}/check`)
+      .reply(200, { allowed: true });
+
+    return { config, tokenScope, apiScope };
+  };
+
+  it('should append /oauth/token when no path is provided', async () => {
+    const { config, tokenScope, apiScope } = testOidcConfig(
+      'auth.example.com',
+      'https://auth.example.com/oauth/token'
+    );
+
+    const client = new OpenFgaClient(config);
+    await client.check({
+      user: 'user:test',
+      relation: 'reader',
+      object: 'document:test'
+    });
+
+    expect(tokenScope.isDone()).toBe(true);
+    expect(apiScope.isDone()).toBe(true);
+  });
+
+  it('should respect custom token paths', async () => {
+    const { config, tokenScope, apiScope } = testOidcConfig(
+      'auth.example.com/oauth/v2/token',
+      'https://auth.example.com/oauth/v2/token'
+    );
+
+    const client = new OpenFgaClient(config);
+    await client.check({
+      user: 'user:test',
+      relation: 'reader',
+      object: 'document:test'
+    });
+
+    expect(tokenScope.isDone()).toBe(true);
+    expect(apiScope.isDone()).toBe(true);
+  });
+
+  it('should handle full URLs with custom paths', async () => {
+    const { config, tokenScope, apiScope } = testOidcConfig(
+      'https://auth.example.com/oauth/v2/token',
+      'https://auth.example.com/oauth/v2/token'
+    );
+
+    const client = new OpenFgaClient(config);
+    await client.check({
+      user: 'user:test',
+      relation: 'reader',
+      object: 'document:test'
+    });
+
+    expect(tokenScope.isDone()).toBe(true);
+    expect(apiScope.isDone()).toBe(true);
+  });
+
+  it('should handle paths with trailing slashes', async () => {
+    const { config, tokenScope, apiScope } = testOidcConfig(
+      'auth.example.com/oauth/v2/',
+      'https://auth.example.com/oauth/v2'
+    );
+
+    const client = new OpenFgaClient(config);
+    await client.check({
+      user: 'user:test',
+      relation: 'reader',
+      object: 'document:test'
+    });
+
+    expect(tokenScope.isDone()).toBe(true);
+    expect(apiScope.isDone()).toBe(true);
+  });
+
+  it('should handle root path correctly', async () => {
+    const { config, tokenScope, apiScope } = testOidcConfig(
+      'auth.example.com/',
+      'https://auth.example.com/oauth/token'
+    );
+
+    const client = new OpenFgaClient(config);
+    await client.check({
+      user: 'user:test',
+      relation: 'reader',
+      object: 'document:test'
+    });
+
+    expect(tokenScope.isDone()).toBe(true);
+    expect(apiScope.isDone()).toBe(true);
+  });
+
+  it('should work with Zitadel-style paths (/oauth/v2/token)', async () => {
+    const { config, tokenScope, apiScope } = testOidcConfig(
+      'https://auth.zitadel.example/oauth/v2/token',
+      'https://auth.zitadel.example/oauth/v2/token'
+    );
+
+    const client = new OpenFgaClient(config);
+    await client.check({
+      user: 'user:test',
+      relation: 'reader',
+      object: 'document:test'
+    });
+
+    expect(tokenScope.isDone()).toBe(true);
+    expect(apiScope.isDone()).toBe(true);
+  });
+
+  it('should work with Entra ID/Azure AD style paths', async () => {
+    const { config, tokenScope, apiScope } = testOidcConfig(
+      'https://login.microsoftonline.com/tenant-id/oauth2/v2.0/token',
+      'https://login.microsoftonline.com/tenant-id/oauth2/v2.0/token'
+    );
+
+    const client = new OpenFgaClient(config);
+    await client.check({
+      user: 'user:test',
+      relation: 'reader',
+      object: 'document:test'
+    });
+
+    expect(tokenScope.isDone()).toBe(true);
+    expect(apiScope.isDone()).toBe(true);
+  });
+});
+```
+
 ### Custom Headers
 
 #### Default Headers

credentials/credentials.ts

@@ -10,7 +10,6 @@
  * NOTE: This file was auto generated by OpenAPI Generator (https://openapi-generator.tech). DO NOT EDIT.
  */
 
-
 import globalAxios, { AxiosInstance } from "axios";
 import * as jose from "jose";
 
@@ -143,21 +142,47 @@ export class Credentials {
    * @return string
    */
   private async refreshAccessToken() {
-    const clientCredentials = (this.authConfig as { method: CredentialsMethod.ClientCredentials; config: ClientCredentialsConfig })?.config;
-    const url = `https://${clientCredentials.apiTokenIssuer}/oauth/token`;
+    const clientCredentials = (this.authConfig as {
+      method: CredentialsMethod.ClientCredentials;
+      config: ClientCredentialsConfig;
+    })?.config;
+    
+    //Normalize the token URL - FIXED VERSION
+    const issuerWithScheme =
+      clientCredentials.apiTokenIssuer.startsWith("http://") ||
+      clientCredentials.apiTokenIssuer.startsWith("https://")
+        ? clientCredentials.apiTokenIssuer
+        : `https://${clientCredentials.apiTokenIssuer}`;
+    
+    let tokenUrl: string;
+    try {
+      const parsed = new URL(issuerWithScheme);
+    
+      // If no path (or just '/'), append the default /oauth/token
+      if (!parsed.pathname || parsed.pathname === "/") {
+        tokenUrl = `${parsed.origin}/oauth/token`;
+      } else {
+        // Otherwise, respect the existing path (e.g. /oauth/v2/token)
+        tokenUrl = parsed.toString().replace(/\/$/, ""); // remove trailing slash if any
+      }
+    } catch {
+      // Fallback to previous behavior if parsing fails
+      tokenUrl = `https://${clientCredentials.apiTokenIssuer}/oauth/token`;
+    }
+    
     const credentialsPayload = await this.buildClientAuthenticationPayload();
-
+    
     try {
-      const wrappedResponse = await attemptHttpRequest<ClientSecretRequest|ClientAssertionRequest, {
-        access_token: string,
-        expires_in: number,
-      }>({
-        url,
+      const wrappedResponse = await attemptHttpRequest<
+        ClientSecretRequest | ClientAssertionRequest,
+        { access_token: string; expires_in: number }
+      >({
+        url: tokenUrl,
         method: "POST",
         data: credentialsPayload,
         headers: {
-          "Content-Type": "application/x-www-form-urlencoded"
-        }
+          "Content-Type": "application/x-www-form-urlencoded",
+        },
       }, {
         maxRetry: 3,
         minWaitInMs: 100,
@@ -170,13 +195,12 @@ export class Credentials {
       }
 
       if (this.telemetryConfig?.metrics?.counterCredentialsRequest?.attributes) {
-
         let attributes = {};
 
         attributes = TelemetryAttributes.fromRequest({
           userAgent: this.baseOptions?.headers["User-Agent"],
           fgaMethod: "TokenExchange",
-          url,
+          url: tokenUrl, // FIXED: Use tokenUrl instead of undefined 'url'
           resendCount: wrappedResponse?.retries,
           httpMethod: "POST",
           credentials: clientCredentials,
@@ -245,4 +269,4 @@ export class Credentials {
 
     throw new FgaValidationError("Credentials method is set to ClientCredentials, but no clientSecret or clientAssertionSigningKey is provided");
   }
-}
+}