fix: use baseURL for OAuth token endpoint rather tnan separate authURL

The searate authURL is no longer used in documentation and not guaranteed
to be supported in the future. This change only impacts TypeScipt integrations
to pre-production SmartThings environments (which require URL overriding).
Integrations to the production environment using default URLs are not affected.

Author: bflorian

src/authenticator.ts

@@ -102,7 +102,7 @@ export class RefreshTokenAuthenticator implements Authenticator {
 		}
 
 		const axiosConfig: AxiosRequestConfig = {
-			url: clientConfig.urlProvider?.authURL,
+			url: `${clientConfig.urlProvider?.baseURL}/oauth/token`,
 			method: 'POST',
 			headers,
 			data: `grant_type=refresh_token&client_id=${refreshData.clientId}&refresh_token=${refreshData.refreshToken}`,

src/endpoint-client.ts

@@ -21,13 +21,11 @@ export type HttpClientMethod =
 
 export interface SmartThingsURLProvider {
 	baseURL: string
-	authURL?: string
 	keyApiURL?: string
 }
 
 export const globalSmartThingsURLProvider: Required<SmartThingsURLProvider> = {
 	baseURL: 'https://api.smartthings.com',
-	authURL: 'https://auth-global.api.smartthings.com/oauth/token',
 	keyApiURL: 'https://key.smartthings.com',
 }
 

src/st-client.ts

@@ -103,7 +103,7 @@ export class SmartThingsOAuthClient {
 
 	constructor(private clientId: string, private clientSecret: string,
 			private redirectUri: string, urlProvider?: SmartThingsURLProvider) {
-		this.authURL = urlProvider?.authURL || globalSmartThingsURLProvider.authURL
+		this.authURL = `${urlProvider?.baseURL || globalSmartThingsURLProvider.baseURL}/oauth/token`
 	}
 
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any

test/unit/authenticator.test.ts

@@ -66,7 +66,7 @@ describe('authenticators', () => {
 
 			expect(axios.request).toHaveBeenCalledTimes(1)
 			expect(axios.request).toHaveBeenCalledWith({
-				'url': 'https://auth-global.api.smartthings.com/oauth/token',
+				'url': 'https://api.smartthings.com/oauth/token',
 				'method': 'POST',
 				'headers': {
 					'Content-Type': 'application/x-www-form-urlencoded',
@@ -128,7 +128,7 @@ describe('authenticators', () => {
 
 				expect(axios.request).toHaveBeenCalledTimes(1)
 				expect(axios.request).toHaveBeenCalledWith({
-					'url': 'https://auth-global.api.smartthings.com/oauth/token',
+					'url': 'https://api.smartthings.com/oauth/token',
 					'method': 'POST',
 					'headers': {
 						'Content-Type': 'application/x-www-form-urlencoded',
@@ -146,7 +146,7 @@ describe('authenticators', () => {
 
 				expect(axios.request).toHaveBeenCalledTimes(1)
 				expect(axios.request).toHaveBeenCalledWith({
-					'url': 'https://auth-global.api.smartthings.com/oauth/token',
+					'url': 'https://api.smartthings.com/oauth/token',
 					'method': 'POST',
 					'headers': {
 						'Content-Type': 'application/x-www-form-urlencoded',

test/unit/signature.test.ts

@@ -44,7 +44,6 @@ describe('ST Padlock', () => {
 		const resolver = new HttpKeyResolver({
 			urlProvider: {
 				baseURL: 'https://api.smartthings.com',
-				authURL: 'https://auth-global.api.smartthings.com/oauth/token',
 				keyApiURL: 'https://keys.smartthingsdev.com',
 			},
 		})