fixing COR-44075 - After the latest axios upgrade from 0.27.2 to 1.6.…

…0 n urls with ? but not params the ? character is omitted

    and we get are reaching a siguration that we are signing https://path/to/somwehere? but sending the request with https://path/to/somewhere

src/ncw-api-client.ts

@@ -11,14 +11,19 @@ export class NcwApiClient implements NcwSdk {
 
     constructor(private readonly apiClient: ApiClient) { }
 
+    public normalizePath(baseUrl: string, params: string): string {
+        return params ? `${baseUrl}?${params.toString()}` : baseUrl;
+    }
+
     public async getSupportedAssets({ pageCursor, pageSize, onlyBaseAssets }: NCW.GetSupportedAssetsPayload): Promise<Web3PagedResponse<NCW.WalletAssetResponse>> {
         const params = new URLSearchParams({
             ...(pageCursor && { pageCursor }),
             ...(pageSize && { pageSize: pageSize.toString() }),
             ...(onlyBaseAssets !== undefined && { onlyBaseAssets: String(onlyBaseAssets) }),
         });
 
-        return await this.apiClient.issueGetRequest(`${this.NCW_BASE_PATH}/supported_assets?${params.toString()}`);
+        const url = this.normalizePath(`${this.NCW_BASE_PATH}/supported_assets`, params.toString());
+        return await this.apiClient.issueGetRequest(url);
     }
 
     public async createWallet(): Promise<{ walletId: string; enabled: boolean; }> {
@@ -77,7 +82,8 @@ export class NcwApiClient implements NcwSdk {
             ...(order && { order }),
         });
 
-        return await this.apiClient.issueGetRequest(`${this.NCW_BASE_PATH}?${params.toString()}`);
+        const url = this.normalizePath(this.NCW_BASE_PATH, params.toString());
+        return await this.apiClient.issueGetRequest(url);
     }
 
     public async getWalletAccounts(walletId: string, { pageCursor, pageSize, sort, order }: NCW.GetWalletsPayload = {}): Promise<Web3PagedResponse<{
@@ -91,8 +97,8 @@ export class NcwApiClient implements NcwSdk {
             ...(order && { order }),
         });
 
-        return await this.apiClient.issueGetRequest(
-            `${this.NCW_BASE_PATH}/${walletId}/accounts?${params.toString()}`);
+        const url = this.normalizePath(`${this.NCW_BASE_PATH}/${walletId}/accounts`, params.toString());
+        return await this.apiClient.issueGetRequest(url);
     }
 
     public async getWalletAccount(walletId: string, accountId: number): Promise<{
@@ -111,8 +117,8 @@ export class NcwApiClient implements NcwSdk {
             ...(order && { order }),
         });
 
-        return await this.apiClient.issueGetRequest(
-            `${this.NCW_BASE_PATH}/${walletId}/accounts/${accountId}/assets?${params.toString()}`);
+        const url = this.normalizePath(`${this.NCW_BASE_PATH}/${walletId}/accounts/${accountId}/assets`, params.toString());
+        return await this.apiClient.issueGetRequest(url);
     }
 
     public async getWalletAsset(walletId: string, accountId: number, assetId: string): Promise<NCW.WalletAssetResponse> {
@@ -132,9 +138,8 @@ export class NcwApiClient implements NcwSdk {
             ...(sort && { sort }),
             ...(order && { order }),
         });
-
-        return await this.apiClient.issueGetRequest(
-            `${this.NCW_BASE_PATH}/${walletId}/accounts/${accountId}/assets/${assetId}/addresses?${params.toString()}`);
+        const url = this.normalizePath(`${this.NCW_BASE_PATH}/${walletId}/accounts/${accountId}/assets/${assetId}/addresses`, params.toString());
+        return await this.apiClient.issueGetRequest(url);
     }
 
     public async getWalletAssetBalance(walletId: string, accountId: number, assetId: string): Promise<AssetResponse> {