Fix the proxy test by stripping the request headers

[absidue]

Pull Request Type

• Bugfix

Related issue

• closes [Bug]: Proxy test endpoint ipwho.is not available #8703

Description

The API we are using for the proxy test button added CORS protection, this pull request strips the request headers from the requests to that API so that the requests still go through.

Testing

I added this patch to the src/renderer/components/ProxySettings/ProxySettings.vue component to force it to show the proxy test button even when a proxy is not configured and clicked the proxy test button.

diff --git a/src/renderer/components/ProxySettings/ProxySettings.vue b/src/renderer/components/ProxySettings/ProxySettings.vue
index 84749afc9..ab34818b4 100644
--- a/src/renderer/components/ProxySettings/ProxySettings.vue
+++ b/src/renderer/components/ProxySettings/ProxySettings.vue
@@ -20,7 +20,7 @@
       />
     </FtFlexBox>
     <template
-      v-if="useProxy"
+      v-if="true"
     >
       <FtFlexBox>
         <FtSelect
@@ -301,12 +301,6 @@ function disableProxy() {
 }
 
 async function testProxy() {
-  isLoading.value = true
-
-  if (!useProxy.value) {
-    enableProxy()
-  }
-
   try {
     const response = await fetch(proxyTestUrl.value)
     const json = await response.json()
@@ -321,10 +315,6 @@ async function testProxy() {
     showToast(t('Settings.Proxy Settings["Error getting network information. Is your proxy configured properly?"]'))
     dataAvailable.value = false
   } finally {
-    if (!useProxy.value) {
-      disableProxy()
-    }
-
     isLoading.value = false
   }
 }

Desktop

• OS: Windows
• OS Version: 11