Use semicolon as value separator instead of comma. The Shibboleth SP separates multi-value SAML2 attributes by semicolons.

northway · northway · commit 8ad820cff9e6 · 2021-03-18T17:17:25.000+01:00
diff --git a/library/Extragroups/UserGroupBackend.php b/library/Extragroups/UserGroupBackend.php
@@ -90,7 +90,7 @@ protected function eventuallyAddExtraGroups($username, & $groups)
 
     protected function splitCommaSeparated($string)
     {
-        return preg_split('/\s*,\s*/', $string, -1, PREG_SPLIT_NO_EMPTY);
+        return preg_split('/\s*;\s*/', $string, -1, PREG_SPLIT_NO_EMPTY);
     }
 
     protected function fillPlaceHolders($string)

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ protected function eventuallyAddExtraGroups($username, & $groups)`
`90`	`90`
`91`	`91`	`protected function splitCommaSeparated($string)`
`92`	`92`	`{`
`93`		`- return preg_split('/\s,\s/', $string, -1, PREG_SPLIT_NO_EMPTY);`
	`93`	`+ return preg_split('/\s;\s/', $string, -1, PREG_SPLIT_NO_EMPTY);`
`94`	`94`	`}`
`95`	`95`
`96`	`96`	`protected function fillPlaceHolders($string)`