[ADF-4810] check publicUrls is array object (#423)

Author: suzanadirla

src/authentication/oauth2Auth.ts

@@ -259,8 +259,11 @@ export class Oauth2Auth extends AlfrescoApiClient {
     isPublicUrl(): boolean {
         const publicUrls = this.config.oauth2.publicUrls || [];
 
-        return publicUrls.length &&
-            publicUrls.some((urlPattern: string) => minimatch(window.location.href, urlPattern));
+        if (Array.isArray(publicUrls)) {
+            return publicUrls.length &&
+                publicUrls.some((urlPattern: string) => minimatch(window.location.href, urlPattern));
+        }
+        return false;
     }
 
     padBase64(base64data: any) {

test/oauth2Auth.spec.ts

@@ -229,6 +229,13 @@ describe('Oauth2  test', function () {
                 expect(this.oauth2Auth.isPublicUrl()).to.be.equal(false);
             });
 
+            it('should return `false` if public urls is not set as an array list', function() {
+                globalAny.window = { location: { href: 'public-url-string'} };
+                this.oauth2Auth.config.oauth2.publicUrls = 'public-url-string';
+
+                expect(this.oauth2Auth.isPublicUrl()).to.be.equal(false);
+            });
+
             it('should match absolute path', function() {
                 globalAny.window = { location: { href: 'http://some-public-url'} };
                 this.oauth2Auth.config.oauth2.publicUrls = ['http://some-public-url'];