Skip to content

Commit

Permalink
fix: remove openid scope, not offered (#546)
Browse files Browse the repository at this point in the history
  • Loading branch information
LoneRifle authored May 22, 2023
1 parent 3764472 commit 31f21e7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/express/sgid.js
Original file line number Diff line number Diff line change
Expand Up @@ -136,9 +136,10 @@ function config(app, { showLoginPage, serviceProvider }) {
.final()
}
const encryptedNric = await encryptPayload(nric)
// sgID doesn't actually offer the openid scope yet
const scopesArr = scopes
.split(' ')
.filter((field) => field !== 'myinfo.nric_number')
.filter((field) => field !== 'openid' && field !== 'myinfo.nric_number')
console.info(`userinfo scopesArr ${scopesArr}`)
const myInfoFields = await Promise.all(
scopesArr.map((scope) =>
Expand Down

0 comments on commit 31f21e7

Please sign in to comment.