feat: release feature flag for proxy support COMPASS-8167 (#6157)

mongodb-js · Aug 27, 2024 · 4eaf037 · 4eaf037
1 parent 63c3e19
commit 4eaf037
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 17 deletions.
diff --git a/packages/compass-connections/src/hooks/use-connection-form-preferences.tsx b/packages/compass-connections/src/hooks/use-connection-form-preferences.tsx
@@ -13,7 +13,6 @@ export function useConnectionFormPreferences() {
   const protectConnectionStringsForNewConnections = usePreference(
     'protectConnectionStringsForNewConnections'
   );
-  const showProxySettings = usePreference('enableProxySupport');
 
   return useMemo(
     () => ({
@@ -24,7 +23,6 @@ export function useConnectionFormPreferences() {
       enableOidc,
       enableDebugUseCsfleSchemaMap,
       protectConnectionStringsForNewConnections,
-      showProxySettings,
     }),
     [
       protectConnectionStrings,
@@ -34,7 +32,6 @@ export function useConnectionFormPreferences() {
       enableOidc,
       enableDebugUseCsfleSchemaMap,
       protectConnectionStringsForNewConnections,
-      showProxySettings,
     ]
   );
 }
diff --git a/packages/compass-e2e-tests/tests/oidc.test.ts b/packages/compass-e2e-tests/tests/oidc.test.ts
@@ -495,7 +495,6 @@ describe('OIDC integration', function () {
 
     beforeEach(async function () {
       await browser.setFeature('proxy', '');
-      await browser.setFeature('enableProxySupport', true);
       httpServer = createHTTPServer();
       ({ connectRequests, httpForwardRequests, connections } =
         setupProxyServer(httpServer));

diff --git a/packages/compass-e2e-tests/tests/proxy.test.ts b/packages/compass-e2e-tests/tests/proxy.test.ts
@@ -95,7 +95,6 @@ describe('Proxy support', function () {
       browser = compass.browser;
 
       await browser.setFeature('proxy', '');
-      await browser.setFeature('enableProxySupport', true);
       httpProxyServer1.removeAllListeners('request');
       ({ connectRequests, connections } = setupProxyServer(httpProxyServer1));
     });

diff --git a/packages/compass-preferences-model/src/feature-flags.ts b/packages/compass-preferences-model/src/feature-flags.ts
@@ -78,7 +78,7 @@ export const featureFlags: Required<{
    * Feature flag for explicit proxy configuration support.
    */
   enableProxySupport: {
-    stage: 'development',
+    stage: 'released',
     description: {
       short: 'Enables support for explicit proxy configuration.',
       long: 'Allows users to specify proxy configuration for the entire Compass application.',

diff --git a/packages/compass-settings/src/components/modal.tsx b/packages/compass-settings/src/components/modal.tsx
@@ -34,7 +34,6 @@ type SettingsModalProps = {
   isAIFeatureEnabled: boolean;
   isOpen: boolean;
   isOIDCEnabled: boolean;
-  isProxySupportEnabled: boolean;
   selectedTab: SettingsTabId | undefined;
   onMount?: () => void;
   onClose: () => void;
@@ -65,7 +64,6 @@ const settingsStyles = css(
 
 export const SettingsModal: React.FunctionComponent<SettingsModalProps> = ({
   isAIFeatureEnabled,
-  isProxySupportEnabled,
   isOpen,
   selectedTab,
   onMount,
@@ -86,6 +84,11 @@ export const SettingsModal: React.FunctionComponent<SettingsModalProps> = ({
     { tabId: 'general', name: 'General', component: GeneralSettings },
     { tabId: 'theme', name: 'Theme', component: ThemeSettings },
     { tabId: 'privacy', name: 'Privacy', component: PrivacySettings },
+    {
+      tabId: 'proxy',
+      name: 'Proxy Configuration',
+      component: ProxySettings,
+    },
   ];
 
   if (
@@ -108,14 +111,6 @@ export const SettingsModal: React.FunctionComponent<SettingsModalProps> = ({
     });
   }
 
-  if (isProxySupportEnabled) {
-    settings.push({
-      tabId: 'proxy',
-      name: 'Proxy Configuration',
-      component: ProxySettings,
-    });
-  }
-
   if (useShouldShowFeaturePreviewSettings()) {
     settings.push({
       tabId: 'preview',
@@ -170,7 +165,6 @@ export default connect(
         state.settings.isModalOpen && state.settings.loadingState === 'ready',
       isAIFeatureEnabled: !!state.settings.settings.enableGenAIFeatures,
       isOIDCEnabled: !!state.settings.settings.enableOidc,
-      isProxySupportEnabled: !!state.settings.settings.enableProxySupport,
       hasChangedSettings: state.settings.updatedFields.length > 0,
       selectedTab: state.settings.tab,
     };