Skip to content

Commit 83bd7bb

Browse files
authored
Merge pull request #1196 from snyk/fix/delete-workload-query-string
fix: delete workload requests send data with query string
2 parents 6e5b679 + 4fc4b05 commit 83bd7bb

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

src/transmitter/index.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,13 +200,17 @@ export async function deleteWorkload(
200200
payload: IDeleteWorkloadPayload,
201201
): Promise<void> {
202202
try {
203+
const { workloadLocator, agentId } = payload;
204+
const { userLocator, cluster, namespace, type, name } = workloadLocator;
205+
const query = `userLocator=${userLocator}&cluster=${cluster}&namespace=${namespace}&type=${type}&name=${name}&agentId=${agentId}`;
203206
const request: KubernetesUpstreamRequest = {
204207
method: 'delete',
205-
url: `${upstreamUrl}/api/v1/workload`,
208+
url: `${upstreamUrl}/api/v1/workload?${query}`,
206209
payload,
207210
};
208211

209212
const { response, attempt } = await reqQueue.pushAsync(request);
213+
// TODO: Remove this check, the upstream no longer returns 404 in such cases
210214
if (response.statusCode === 404) {
211215
logger.info(
212216
{ payload },

0 commit comments

Comments
 (0)