Skip to content

Commit

Permalink
fix: send broker version and right api cfg var
Browse files Browse the repository at this point in the history
  • Loading branch information
aarlaud committed Jan 30, 2025
1 parent 5067cec commit 14a63c7
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion lib/client/socket.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import { getServerId } from './dispatcher';
import { determineFilterType } from './utils/filterSelection';
import { notificationHandler } from './socketHandlers/notificationHandler';
import { renewBrokerServerConnection } from './auth/brokerServerConnection';
import version from '../common/utils/version';

export const createWebSocketConnectionPairs = async (
websocketConnections: WebSocketConnection[],
Expand Down Expand Up @@ -139,13 +140,15 @@ export const createWebSocket = (
pong: parseInt(localClientOps.config.socketPongTimeout) || 10000,
timeout: parseInt(localClientOps.config.socketConnectTimeout) || 10000,
};

console.log('@@@@@@@@@@@@@@@@@@@@@2');
console.log(clientOpts.accessToken?.authHeader);
if (clientOpts.accessToken && clientOpts.config.UNIVERSAL_BROKER_GA) {
socketSettings['transport'] = {
extraHeaders: {
Authorization: clientOpts.accessToken?.authHeader,
'x-snyk-broker-client-id': identifyingMetadata.clientId,
'x-snyk-broker-client-role': identifyingMetadata.role,
'x-broker-client-version': version,
},
};
}
Expand Down Expand Up @@ -183,6 +186,7 @@ export const createWebSocket = (
Authorization: clientOpts.accessToken!.authHeader,
'x-snyk-broker-client-id': identifyingMetadata.clientId,
'x-snyk-broker-client-role': identifyingMetadata.role,
'x-broker-client-version': version,
};

logger.debug(
Expand Down
2 changes: 1 addition & 1 deletion lib/hybrid-sdk/http/downstream-post-stream-to-server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ class BrokerServerPostResponseHandler {
try {
const backendHostname =
this.#config.universalBrokerEnabled && this.#config.universalBrokerGa
? `${this.#config.API_BASE_URL}/hidden/broker`
? `${this.#config.apiHostname}/hidden/broker`
: this.#config.brokerServerUrl;
const url = new URL(
`${backendHostname}/response-data/${this.#brokerToken}/${
Expand Down

0 comments on commit 14a63c7

Please sign in to comment.