fix(App): Use system proxy for services

meetfranz · Dec 4, 2018 · 8ddae4a · 8ddae4a
1 parent 5c18595
commit 8ddae4a
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 17 deletions.
diff --git a/src/config.js b/src/config.js
@@ -10,6 +10,8 @@ export const DEV_API = 'https://dev.franzinfra.com';
 export const LIVE_API = 'https://api.franzinfra.com';
 export const GA_ID = 'UA-74126766-10';
 
+export const HEALTHCHECK_URL = 'https://api.franzinfra.com/health';
+
 export const DEFAULT_APP_SETTINGS = {
   autoLaunchInBackground: false,
   runInBackground: true,

diff --git a/src/features/serviceProxy/index.js b/src/features/serviceProxy/index.js
@@ -1,7 +1,7 @@
 import { autorun, observable } from 'mobx';
 import { remote } from 'electron';
 
-import { DEFAULT_FEATURES_CONFIG } from '../../config';
+import { DEFAULT_FEATURES_CONFIG, HEALTHCHECK_URL } from '../../config';
 
 const { session } = remote;
 
@@ -15,29 +15,40 @@ export const config = observable({
 export default function init(stores) {
   debug('Initializing `serviceProxy` feature');
 
-  autorun(() => {
+  const { defaultSession } = session;
-    const { isServiceProxyEnabled, isServiceProxyPremiumFeature } = stores.features.features;
+  defaultSession.resolveProxy(HEALTHCHECK_URL, (proxy) => {
+    debug('Resolve proxy', proxy);
 
-    config.isEnabled = isServiceProxyEnabled !== undefined ? isServiceProxyEnabled : DEFAULT_FEATURES_CONFIG.isServiceProxyEnabled;
+    let systemProxy = 'direct://';
-    config.isPremium = isServiceProxyPremiumFeature !== undefined ? isServiceProxyPremiumFeature : DEFAULT_FEATURES_CONFIG.isServiceProxyPremiumFeature;
+    const proxyMatch = proxy.match(/PROXY (.*)/i);
+    if (proxyMatch.length) {
+      systemProxy = proxyMatch[1].trim();
+    }
 
-    const services = stores.services.all;
+    autorun(() => {
-    const isPremiumUser = stores.user.data.isPremium;
+      const { isServiceProxyEnabled, isServiceProxyPremiumFeature } = stores.features.features;
 
-    services.forEach((service) => {
+      config.isEnabled = isServiceProxyEnabled !== undefined ? isServiceProxyEnabled : DEFAULT_FEATURES_CONFIG.isServiceProxyEnabled;
-      const s = session.fromPartition(`persist:service-${service.id}`);
+      config.isPremium = isServiceProxyPremiumFeature !== undefined ? isServiceProxyPremiumFeature : DEFAULT_FEATURES_CONFIG.isServiceProxyPremiumFeature;
-      let proxyHost = 'direct://';
 
-      if (config.isEnabled && (isPremiumUser || !config.isPremium)) {
+      const services = stores.services.enabled;
-        const serviceProxyConfig = stores.settings.proxy[service.id];
+      const isPremiumUser = stores.user.data.isPremium;
 
-        if (serviceProxyConfig && serviceProxyConfig.isEnabled && serviceProxyConfig.host) {
+      services.forEach((service) => {
-          proxyHost = serviceProxyConfig.host;
+        let proxyHost = systemProxy;
+        const s = session.fromPartition(`persist:service-${service.id}`);
+
+        if (config.isEnabled && (isPremiumUser || !config.isPremium)) {
+          const serviceProxyConfig = stores.settings.proxy[service.id];
+
+          if (serviceProxyConfig && serviceProxyConfig.isEnabled && serviceProxyConfig.host) {
+            proxyHost = serviceProxyConfig.host;
+          }
         }
-      }
 
-      s.setProxy({ proxyRules: proxyHost }, (e) => {
+        s.setProxy({ proxyRules: proxyHost }, () => {
-        debug(`Using proxy "${proxyHost}" for "${service.name}" (${service.id})`, e);
+          debug(`Using proxy "${proxyHost}" for "${service.name}" (${service.id})`);
+        });
       });
     });
   });