Skip to content

Commit b808b61

Browse files
committed
chore: improve oidc error handling
1 parent e77aec2 commit b808b61

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

src/service/passport/oidc.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,16 @@ const configure = async (passport) => {
1717
}
1818

1919
const server = new URL(issuer);
20+
let config;
2021

2122
try {
22-
const config = await discovery(server, clientID, clientSecret);
23+
config = await discovery(server, clientID, clientSecret);
24+
} catch (error) {
25+
console.error('Error during OIDC discovery:', error);
26+
throw new Error('OIDC setup error (discovery): ' + error.message);
27+
}
2328

29+
try {
2430
const strategy = new Strategy({ callbackURL, config, scope }, async (tokenSet, done) => {
2531
// Validate token sub for added security
2632
const idTokenClaims = tokenSet.claims();
@@ -56,8 +62,8 @@ const configure = async (passport) => {
5662

5763
return passport;
5864
} catch (error) {
59-
console.error('OIDC configuration failed:', error);
60-
throw error;
65+
console.error('Error during OIDC passport setup:', error);
66+
throw new Error('OIDC setup error (strategy): ' + error.message);
6167
}
6268
};
6369

0 commit comments

Comments
 (0)