Merge pull request #2 from isotoma/NGW-1086

NGW-1086  LinkedIn SSO - doesn't populate name even though it's avail…

Author: 9080tom

src/linkedin.js

@@ -14,7 +14,7 @@ const getApiEndpoints = (
   apiBaseUrl = LINKEDIN_API_URL,
   loginBaseUrl = LINKEDIN_LOGIN_URL
 ) => ({
-  userDetails: `${apiBaseUrl}/v2/me`,
+  userDetails: `${apiBaseUrl}/v2/me?projection=(id,localizedLastName,localizedFirstName,profilePicture(displayImage~:playableStreams))`,
   userEmails: `${apiBaseUrl}/v2/clientAwareMemberHandles?q=members&projection=(elements*(primary,type,handle~))`,
   oauthToken: `${apiBaseUrl}/oauth/v2/accessToken`,
   oauthAuthorize: `${loginBaseUrl}/oauth/v2/authorization`,
@@ -40,7 +40,7 @@ const check = response => {
   );
 };
 
-const linkedinGet = (url, accessToken) =>
+  const linkedinGet = (url, accessToken) =>
   axios({
     method: 'get',
     url,
@@ -59,7 +59,7 @@ module.exports = (apiBaseUrl, loginBaseUrl) => {
       )}&state=${state}&response_type=${response_type}&redirect_uri=${COGNITO_REDIRECT_URI}`;
     },
     getUserDetails: accessToken =>
-      linkedinGet(urls.userDetails, accessToken).then(check),
+    linkedinGet(urls.userDetails, accessToken).then(check),
     getUserEmails: accessToken =>
       linkedinGet(urls.userEmails, accessToken).then(check),
     getToken: (code, state) => {

src/openid.js

@@ -16,9 +16,12 @@ const getUserInfo = accessToken =>
         // https://docs.microsoft.com/en-us/linkedin/shared/integrations/people/profile-api?context=linkedin/consumer/context
         // and
         // https://docs.microsoft.com/en-us/linkedin/shared/references/v2/profile/lite-profile
+        const picture = userDetails.profilePicture["displayImage~"]?.elements[1]?.identifiers[0]?.identifier
         const claims = {
           sub: `${userDetails.id}`, // OpenID requires a string
-          name: `${userDetails.firstName.localized} ${userDetails.lastName.localized}`,
+          firstName: `${userDetails.localizedFirstName}`,
+          lastName: `${userDetails.localizedLastName}`,
+          picture: picture,
         };
         logger.debug('Resolved claims: %j', claims, {});
         return claims;
@@ -113,7 +116,9 @@ const getConfigFor = host => ({
   display_values_supported: ['page', 'popup'],
   claims_supported: [
     'sub',
-    'name',
+    'firstName',
+    'lastName',
+    'picture',
     'email',
     'email_verified',
   ]