Merge pull request #50 from nodeSolidServer/fix/issue#48

Scope normalization logic that converts array scope parameters to space-separated strings before validation

Author: bourgeoa

src/AuthenticationRequest.js

@@ -50,6 +50,11 @@ class AuthenticationRequest {
         client = { client_id: registration.client_id}
         params = Object.assign(defaults.authenticate, client, options)
 
+        // normalize scope parameter - convert array to space-separated string if needed
+        if (Array.isArray(params.scope)) {
+          params.scope = params.scope.join(' ')
+        }
+
         // validate presence of required configuration and parameters
         assert(issuer,
           'Missing issuer in provider OpenID Configuration')

test/AuthenticationRequestSpec.js

@@ -182,6 +182,13 @@ describe('AuthenticationRequest', () => {
       })
     })
 
+    it('should handle array scope parameters', () => {
+      options = { scope: ['openid', 'webid', 'profile'] }
+      return AuthenticationRequest.create(rp, options, session).then(url => {
+        url.should.include('scope=openid%20webid%20profile')
+      })
+    })
+
     it('should set `client_id` parameter', () => {
       return AuthenticationRequest.create(rp, options, session).then(url => {
         url.should.include('client_id=uuid')