Skip to content

Commit 3847d64

Browse files
authored
Merge pull request #4933 from Infisical/better-k8s-auth-logs
chore: better k8s auth error logging
2 parents 8722f65 + edd8e61 commit 3847d64

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

backend/src/services/identity-kubernetes-auth/identity-kubernetes-auth-service.ts

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,13 @@ export const identityKubernetesAuthServiceFactory = ({
270270
}
271271
)
272272
.catch((err) => {
273+
const tokenReviewerJwtSnippet = `${tokenReviewerJwt?.substring?.(0, 10) || ""}...${tokenReviewerJwt?.substring?.(tokenReviewerJwt.length - 10) || ""}`;
274+
const serviceAccountJwtSnippet = `${serviceAccountJwt?.substring?.(0, 10) || ""}...${serviceAccountJwt?.substring?.(serviceAccountJwt.length - 10) || ""}`;
273275
if (err instanceof AxiosError) {
276+
logger.error(
277+
{ response: err.response, host, port, tokenReviewerJwtSnippet, serviceAccountJwtSnippet },
278+
"tokenReviewCallbackRaw: Kubernetes token review request error (request error)"
279+
);
274280
if (err.response) {
275281
const { message } = err?.response?.data as unknown as { message?: string };
276282

@@ -281,6 +287,11 @@ export const identityKubernetesAuthServiceFactory = ({
281287
});
282288
}
283289
}
290+
} else {
291+
logger.error(
292+
{ error: err as Error, host, port, tokenReviewerJwtSnippet, serviceAccountJwtSnippet },
293+
"tokenReviewCallbackRaw: Kubernetes token review request error (non-request error)"
294+
);
284295
}
285296
throw err;
286297
});

0 commit comments

Comments
 (0)