Assume multiple UDDS values are delimited by ;

src/main/edu/wisc/doit/uwphps/UWUserDetails.php

@@ -14,7 +14,7 @@ class UWUserDetails implements UserDetails
     private $pvi;
     /** @var string */
     private $fullName;
-    /** @var string[] */
+    /** @var array */
     private $udds;
     /** @var string */
     private $emailAddress;
@@ -32,7 +32,7 @@ class UWUserDetails implements UserDetails
      * @param string $eppn
      * @param string $pvi
      * @param string $fullName
-     * @param string[] $udds
+     * @param array $udds
      * @param string $emailAddress
      * @param string $source
      * @param string $isisEmplid

src/main/edu/wisc/doit/uwphps/UserDetails.php

@@ -34,7 +34,7 @@ interface UserDetails
     /**
      * A never null, but possibly empty, array of strings each representing the UDDS IDs of the group this user is a member of.
      * 
-     * @return string[]
+     * @return array
      */
     public function getUddsMembership();
 

src/main/edu/wisc/doit/uwphps/UserDetailsProvider.php

@@ -20,6 +20,9 @@ abstract class UserDetailsProvider
     const SOURCE = "source";
     const ISIS_EMPLID = "isisEmplid";
 
+    // Delimiter used by multi-valued headers
+    const DELIMITER = ';';
+
     /**
      * UserDetailsProvider constructor.
      * @param $http

src/main/edu/wisc/doit/uwphps/local/LocalUserDetailsProvider.php

@@ -41,7 +41,7 @@ class LocalUserDetailsProvider extends PreauthUserDetailsProvider
             $attributes[static::EPPN],
             $attributes[static::PVI],
             $attributes[static::FULL_NAME],
-            $attributes[static::UDDS],
+            explode(static::DELIMITER, $attributes[static::UDDS]),
             $attributes[static::EMAIL],
             $attributes[static::SOURCE],
             $attributes[static::ISIS_EMPLID],

src/main/edu/wisc/doit/uwphps/preauth/FederatedPreauthUserDetailsProvider.php

@@ -37,7 +37,7 @@ class FederatedPreauthUserDetailsProvider extends UserDetailsProvider
                 getenv($this->mapAttribute(static::EPPN)),
                 getenv($this->mapAttribute(static::SPVI)),
                 getenv($this->mapAttribute(static::FULL_NAME)),
-                getenv($this->mapAttribute(static::UDDS)),
+                explode(static::DELIMITER, getenv($this->mapAttribute(static::UDDS))),
                 getenv($this->mapAttribute(static::EMAIL)),
                 getenv($this->mapAttribute(static::SOURCE)),
                 getenv($this->mapAttribute(static::ISIS_EMPLID)),

src/main/edu/wisc/doit/uwphps/preauth/PreauthUserDetailsProvider.php

@@ -39,7 +39,7 @@ class PreauthUserDetailsProvider extends UserDetailsProvider
                 getenv($this->mapAttribute(static::EPPN)),
                 getenv($this->mapAttribute(static::PVI)),
                 getenv($this->mapAttribute(static::FULL_NAME)),
-                getenv($this->mapAttribute(static::UDDS)),
+                explode(static::DELIMITER, getenv($this->mapAttribute(static::UDDS))),
                 getenv($this->mapAttribute(static::EPPN)),
                 getenv($this->mapAttribute(static::SOURCE)),
                 getenv($this->mapAttribute(static::ISIS_EMPLID)),

src/main/resources/localuser.json

@@ -5,10 +5,7 @@
   "cn": "BUCKINGHAM BADGER",
   "givenName": "BUCKINGHAM",
   "sn": "BADGER",
-  "wiscEduUDDS": [
-    "UW123A456",
-    "UW234A567"
-  ],
+  "wiscEduUDDS": "A061234;A072345",
   "mail": "bucky.badger@wisc.edu",
   "source": "a_source",
   "isisEmplid": "123456789"

src/test/edu/wisc/doit/uwphps/preauth/HTTPFederatedPreauthUserDetailsProviderTest.php

@@ -33,7 +33,7 @@ class HTTPFederatedPreauthUserDetailsProviderTest extends \PHPUnit_Framework_Tes
         putenv($this->mapAttribute(FederatedPreauthUserDetailsProvider::FULL_NAME) . '=' . $this->attributes[$this->mapAttribute(FederatedPreauthUserDetailsProvider::FULL_NAME)]);
         putenv($this->mapAttribute(FederatedPreauthUserDetailsProvider::FIRST_NAME) . '=' . $this->attributes[$this->mapAttribute(FederatedPreauthUserDetailsProvider::FIRST_NAME)]);
         putenv($this->mapAttribute(FederatedPreauthUserDetailsProvider::LAST_NAME) . '=' . $this->attributes[$this->mapAttribute(FederatedPreauthUserDetailsProvider::LAST_NAME)]);
-        putenv($this->mapAttribute(UserDetailsProvider::UDDS) . '=' . implode(",", $this->attributes[$this->mapAttribute(UserDetailsProvider::UDDS)]));
+        putenv($this->mapAttribute(UserDetailsProvider::UDDS) . '=' . $this->attributes[$this->mapAttribute(UserDetailsProvider::UDDS)]);
         putenv($this->mapAttribute(FederatedPreauthUserDetailsProvider::EMAIL) . '=' . $this->attributes[$this->mapAttribute(FederatedPreauthUserDetailsProvider::EMAIL)]);
         putenv($this->mapAttribute(UserDetailsProvider::SOURCE) . '=' . $this->attributes[$this->mapAttribute(UserDetailsProvider::SOURCE)]);
         putenv($this->mapAttribute(UserDetailsProvider::ISIS_EMPLID) . '=' . $this->attributes[$this->mapAttribute(UserDetailsProvider::ISIS_EMPLID)]);
@@ -55,6 +55,7 @@ class HTTPFederatedPreauthUserDetailsProviderTest extends \PHPUnit_Framework_Tes
         $this->assertEquals("123456789", $user->getIsisEmplid());
         $this->assertEquals("BUCKINGHAM", $user->getFirstName());
         $this->assertEquals("BADGER", $user->getLastName());
+        $this->assertEquals(["A061234", "A072345"], $user->getUddsMembership());
     }
 
     /**

src/test/resources/localuser.json

@@ -5,10 +5,7 @@
   "cn": "BUCKINGHAM BADGER",
   "givenName": "BUCKINGHAM",
   "sn": "BADGER",
-  "wiscEduUDDS": [
-    "UW123A456",
-    "UW234A567"
-  ],
+  "wiscEduUDDS": "A061234;A072345",
   "mail": "bucky.badger@wisc.edu",
   "source": "a_source",
   "wiscEduIsisEmplid": "123456789"

src/test/resources/testuser_http.json

@@ -5,10 +5,7 @@
   "HTTP_EDUWISCONSINCOMMONNAME": "BUCKINGHAM BADGER",
   "HTTP_EDUWISCONSINGIVENNAME": "BUCKINGHAM",
   "HTTP_EDUWISCONSINSURNAME": "BADGER",
-  "HTTP_UDDS": [
-    "UW123A456",
-    "UW234A567"
-  ],
+  "HTTP_UDDS": "A061234;A072345",
   "HTTP_EDUWISCONSINEMAILADDRESS": "bucky.badger@wisc.edu",
   "HTTP_SOURCE": "a_source",
   "HTTP_ISISEMPLID": "123456789",