fix: introduce openid scope (#544)

* fix(scopes): add openid * fix(sgid): properly include openid scope
opengovsg · May 18, 2023 · c90632e · c90632e
1 parent 15a4062
commit c90632e
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/express/sgid.js b/lib/express/sgid.js
@@ -138,7 +138,7 @@ function config(app, { showLoginPage, serviceProvider }) {
     const encryptedNric = await encryptPayload(nric)
     const scopesArr = scopes
       .split(' ')
-      .filter((field) => field !== 'openid' && field !== 'myinfo.nric_number')
+      .filter((field) => field !== 'myinfo.nric_number')
     console.info(`userinfo scopesArr ${scopesArr}`)
     const myInfoFields = await Promise.all(
       scopesArr.map((scope) =>
@@ -236,6 +236,8 @@ const concatMyInfoRegAddr = (regadd) => {
 const sgIDScopeToMyInfoField = (persona, scope) => {
   switch (scope) {
     // No NRIC as that is always returned by default
+    case 'openid':
+      return persona.uuid.value
     case 'myinfo.name':
       return persona.name.value
     case 'myinfo.email':