[MM-61821] Automatically allow permission checks for supported permission types through for GPO configured servers

src/main/permissionsManager.test.js

@@ -3,6 +3,7 @@
 
 import {dialog, systemPreferences} from 'electron';
 
+import Config from 'common/config';
 import {parseURL, isTrustedURL} from 'common/utils/url';
 import ViewManager from 'main/views/viewManager';
 import CallsWidgetWindow from 'main/windows/callsWidgetWindow';
@@ -37,6 +38,12 @@
     isTrustedURL: jest.fn(),
 }));
 
+jest.mock('common/config', () => ({
+    registryData: {
+        servers: []
+    },
+}));
+
 jest.mock('main/i18nManager', () => ({
     localizeMessage: jest.fn(),
 }));
@@ -72,6 +79,9 @@
                 if (id === 2) {
                     return {view: {server: {url: new URL('http://anyurl.com')}}};
                 }
+                if (id === 4) {
+                    return {view: {server: {url: new URL('http://gposerver.com')}}};
+                }
 
                 return null;
             });
@@ -84,6 +94,11 @@
                 }
             });
             isTrustedURL.mockImplementation((url, baseURL) => url.toString().startsWith(baseURL.toString()));
+            Config.registryData.servers = [
+                {
+                    url: 'http://gposerver.com',
+                }
+            ];
         });
 
         afterEach(() => {
@@ -115,10 +130,17 @@
         it('should deny if the server URL can not be found', async () => {
             const permissionsManager = new PermissionsManager('anyfile.json');
             const cb = jest.fn();
-            await permissionsManager.handlePermissionRequest({id: 4}, 'media', cb, {securityOrigin: 'http://anyurl.com'});
+            await permissionsManager.handlePermissionRequest({id: 5}, 'media', cb, {securityOrigin: 'http://anyurl.com'});
             expect(cb).toHaveBeenCalledWith(false);
         });
 
+        it('should allow if the URL is a GPO configured server', async () => {
+            const permissionsManager = new PermissionsManager('anyfile.json');
+            const cb = jest.fn();
+            await permissionsManager.handlePermissionRequest({id: 4}, 'media', cb, {securityOrigin: 'http://gposerver.com'});
+            expect(cb).toHaveBeenCalledWith(true);
+        });
+
         it('should deny if the URL is not trusted', async () => {
             const permissionsManager = new PermissionsManager('anyfile.json');
             const cb = jest.fn();

src/main/permissionsManager.ts

@@ -22,6 +22,7 @@ import {
     OPEN_WINDOWS_MICROPHONE_PREFERENCES,
     UPDATE_PATHS,
 } from 'common/communication';
+import Config from 'common/config';
 import JsonFileManager from 'common/JsonFileManager';
 import {Logger} from 'common/log';
 import type {MattermostServer} from 'common/servers/MattermostServer';
@@ -141,7 +142,7 @@ export class PermissionsManager extends JsonFileManager<PermissionsByOrigin> {
             return false;
         }
 
-        let serverURL;
+        let serverURL: URL | undefined;
         if (CallsWidgetWindow.isCallsWidget(webContentsId)) {
             serverURL = CallsWidgetWindow.getViewURL();
         } else {
@@ -152,6 +153,11 @@ export class PermissionsManager extends JsonFileManager<PermissionsByOrigin> {
             return false;
         }
 
+        // For GPO servers, we always allow permissions since they are trusted
+        if (Config.registryData?.servers?.some((s) => parseURL(s.url)?.href === serverURL.href)) {
+            return true;
+        }
+
         // Exception for embedded videos such as YouTube
         // We still want to ask permission to do this though
         const isExternalFullscreen = permission === 'fullscreen' && parsedURL.origin !== serverURL.origin;