From 9b5ed176cddffead98e264d1b532a6ed361c7bce Mon Sep 17 00:00:00 2001
From: Yen Chi Hsuan <yan12125@gmail.com>
Date: Sat, 6 Jan 2018 18:22:10 +0800
Subject: [PATCH] Improve proxy path handling for the browser plugin

If "Update KeePassXC binary path automatically to native messaging
scripts on startup" is not selected (default behavior), native messaging
host JSON contains wrong path " ".
---
 src/browser/BrowserSettings.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/browser/BrowserSettings.cpp b/src/browser/BrowserSettings.cpp
index d15f57d46..2df7fc1c5 100755
--- a/src/browser/BrowserSettings.cpp
+++ b/src/browser/BrowserSettings.cpp
@@ -156,7 +156,10 @@ void BrowserSettings::setUseCustomProxy(bool enabled)
 
 QString BrowserSettings::customProxyLocation()
 {
-    return config()->get("Browser/CustomProxyLocation", " ").toString();
+    if (!useCustomProxy()) {
+        return QString();
+    }
+    return config()->get("Browser/CustomProxyLocation", "").toString();
 }
 
 void BrowserSettings::setCustomProxyLocation(QString location)