fix

src/server/routes/login-passport.js

@@ -326,9 +326,9 @@ module.exports = function(crowi, app) {
     const strategyName = 'saml';
     const response = await promisifiedPassportAuthentication(req, res, next, strategyName);
     const userInfo = {
-      'id': response.id,
-      'username': response.username,
-      'name': `${response.firstName} ${response.lastName}`,
+      'id': response[config.crowi['security:passport-saml:attrMapId']],
+      'username': response[config.crowi['security:passport-saml:attrMapUsername']],
+      'name': `${response[config.crowi['security:passport-saml:attrMapFirstName']]} ${response[config.crowi['security:passport-saml:attrMapLastName']]}`,
     };
 
     const externalAccount = await getOrCreateUser(req, res, next, userInfo, providerId);

src/server/views/admin/widget/passport/saml.html

@@ -105,19 +105,6 @@
       </div>
     </div>
 
-    <div class="form-group">
-      <label for="settingForm[security:passport-saml:attrMapMail]" class="col-xs-3 control-label">Mail</label>
-      <div class="col-xs-6">
-        <input class="form-control" type="text" placeholder="Default: mail"
-            name="settingForm[security:passport-saml:attrMapMail]" value="{{ settingForm['security:passport-saml:attrMapMail'] || '' }}">
-        <p class="help-block">
-          <small>
-            {{ t("security_setting.SAML.mapping_detail", "Email") }}
-          </small>
-        </p>
-      </div>
-    </div>
-
     <div class="form-group">
       <label for="settingForm[security:passport-saml:attrMapFirstName]" class="col-xs-3 control-label">First Name</label>
       <div class="col-xs-6">