Răsfoiți Sursa

Merge pull request #2250 from overleaf/jel-saml-session-email

Non canonical institution email in session

GitOrigin-RevId: b70971f6f42391391a42ba6ff214c14a275461d3
Jessica Lawshe 7 ani în urmă
părinte
comite
34c04660a6

+ 5 - 0
services/web/app/src/Features/User/UserPagesController.js

@@ -117,6 +117,10 @@ const UserPagesController = {
     // Institution SSO
     // Institution SSO
     const institutionLinked = _.get(req.session, ['saml', 'linked'])
     const institutionLinked = _.get(req.session, ['saml', 'linked'])
     const institutionNotLinked = _.get(req.session, ['saml', 'notLinked'])
     const institutionNotLinked = _.get(req.session, ['saml', 'notLinked'])
+    const institutionEmailNonCanonical = _.get(req.session, [
+      'saml',
+      'emailNonCanonical'
+    ])
     delete req.session.saml
     delete req.session.saml
     logger.log({ user: userId }, 'loading settings page')
     logger.log({ user: userId }, 'loading settings page')
     let shouldAllowEditingDetails = true
     let shouldAllowEditingDetails = true
@@ -146,6 +150,7 @@ const UserPagesController = {
         oauthUseV2: Settings.oauthUseV2 || false,
         oauthUseV2: Settings.oauthUseV2 || false,
         institutionLinked,
         institutionLinked,
         institutionNotLinked,
         institutionNotLinked,
+        institutionEmailNonCanonical,
         samlBeta: req.session.samlBeta,
         samlBeta: req.session.samlBeta,
         ssoError: ssoError,
         ssoError: ssoError,
         thirdPartyIds: UserPagesController._restructureThirdPartyIds(user)
         thirdPartyIds: UserPagesController._restructureThirdPartyIds(user)

+ 2 - 2
services/web/app/views/user/settings/user-affiliations.pug

@@ -223,7 +223,7 @@ form.row(
 							.small !{translate("institution_acct_successfully_linked", {institutionName: institutionLinked.universityName})}
 							.small !{translate("institution_acct_successfully_linked", {institutionName: institutionLinked.universityName})}
 							if institutionLinked.hasEntitlement
 							if institutionLinked.hasEntitlement
 								.small !{translate("this_grants_access_to_features", {featureType: translate("professional")})}
 								.small !{translate("this_grants_access_to_features", {featureType: translate("professional")})}
-					if institutionLinked.emailViaInstitution
+					if institutionEmailNonCanonical
 						tr.affiliations-table-warning-row(ng-if="!hideInstitutionNotifications.warning")
 						tr.affiliations-table-warning-row(ng-if="!hideInstitutionNotifications.warning")
 							td(colspan="3").text-center(aria-live="assertive")
 							td(colspan="3").text-center(aria-live="assertive")
 								button.close(
 								button.close(
@@ -236,7 +236,7 @@ form.row(
 								.small
 								.small
 									i.fa.fa-exclamation-triangle(aria-hidden="true")
 									i.fa.fa-exclamation-triangle(aria-hidden="true")
 									|  
 									|  
-									| !{translate("in_order_to_match_institutional_metadata", {email: institutionLinked.emailViaInstitution})}
+									| !{translate("in_order_to_match_institutional_metadata", {email: institutionEmailNonCanonical})}
 				if institutionNotLinked
 				if institutionNotLinked
 					tr.affiliations-table-error-row(ng-if="!hideInstitutionNotifications.error")
 					tr.affiliations-table-error-row(ng-if="!hideInstitutionNotifications.error")
 						td(colspan="3").text-center(aria-live="assertive")
 						td(colspan="3").text-center(aria-live="assertive")