diff --git a/packages/security/oidc-provider/src/services/OidcProvider.spec.ts b/packages/security/oidc-provider/src/services/OidcProvider.spec.ts index 3ffbf609c1a..7bdee31ebab 100644 --- a/packages/security/oidc-provider/src/services/OidcProvider.spec.ts +++ b/packages/security/oidc-provider/src/services/OidcProvider.spec.ts @@ -39,6 +39,9 @@ describe("OidcProvider", () => { const oidcProvider = PlatformTest.get(OidcProvider); jest.spyOn((oidcProvider as any).injector.logger, "error"); + const error = new Error("test"); + Object.assign(error, {error: "error", error_description: "error_description", error_detail: "error_detail"}); + const fn = (oidcProvider as any).createErrorHandler("event"); fn( { @@ -51,14 +54,20 @@ describe("OidcProvider", () => { } } }, - {error: "error", error_description: "error_description", error_detail: "error_detail"}, + error, "account_id", "sid" ); expect((oidcProvider as any).injector.logger.error).toHaveBeenCalledWith({ account_id: "account_id", - error: {error_description: "error_description", error_detail: "error_detail", error: "error"}, + error: { + message: "test", + stack: error.stack, + error_description: "error_description", + error_detail: "error_detail", + error: "error" + }, event: "OIDC_ERROR", headers: { origin: "origin" diff --git a/packages/security/oidc-provider/src/services/OidcProvider.ts b/packages/security/oidc-provider/src/services/OidcProvider.ts index 1402e103ecd..a15e547dc31 100644 --- a/packages/security/oidc-provider/src/services/OidcProvider.ts +++ b/packages/security/oidc-provider/src/services/OidcProvider.ts @@ -13,12 +13,17 @@ import {OidcInteractions} from "./OidcInteractions"; import {OidcJwks} from "./OidcJwks"; function mapError(error: any) { - return Object.getOwnPropertyNames(error).reduce((obj: any, key) => { - return { - ...obj, - [key]: error[key] - }; - }, {}); + return Object.getOwnPropertyNames(error).reduce( + (obj: any, key) => { + return { + ...obj, + [key]: error[key] + }; + }, + { + stack: error.stack + } + ); } @Injectable()