Merge pull request #106 from dpoulson/master

Allow default COMMON_NAME field to be configurable
codegreencreative · Dec 1, 2022 · ba23dd7 · ba23dd7
2 parents c0dcc98 + 2521228
commit ba23dd7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/config/samlidp.php b/config/samlidp.php
@@ -13,6 +13,8 @@
     'debug' => false,
     // Define the email address field name in the users table
     'email_field' => 'email',
+    // Define the name field in the users table
+    'name_field' => 'name',
     // The URI to your login page
     'login_uri' => 'login',
     // Log out of the IdP after SLO

diff --git a/src/Events/Assertion.php b/src/Events/Assertion.php
@@ -30,6 +30,6 @@ public function __construct(\LightSaml\Model\Assertion\AttributeStatement &$attr
         $this->attribute_statement = &$attribute_statement;
         $this->attribute_statement
             ->addAttribute(new Attribute(ClaimTypes::EMAIL_ADDRESS, auth($guard)->user()->__get(config('samlidp.email_field', 'email'))))
-            ->addAttribute(new Attribute(ClaimTypes::COMMON_NAME, auth($guard)->user()->name));
+            ->addAttribute(new Attribute(ClaimTypes::COMMON_NAME, auth($guard)->user()->__get(config('samlidp.name_field', 'name'))));
     }
 }