Merge pull request #853 from Thijn/main

fix: Automatically prepend https for Overseerr, Sonarr and Radarr when using port 443
jorenn92 · Mar 25, 2024 · 995d01e · 995d01e
2 parents e0c4dca + 1616cfd
commit 995d01e
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 5 deletions.
diff --git a/ui/src/components/Settings/Overseerr/index.tsx b/ui/src/components/Settings/Overseerr/index.tsx
@@ -79,7 +79,10 @@ const OverseerrSettings = () => {
         ? hostnameRef.current.value
         : hostnameRef.current.value.includes('https://')
           ? hostnameRef.current.value
-          : 'http://' + hostnameRef.current.value
+        : portRef.current.value == '443'
+        ? 'https://' + hostnameRef.current.value
+        : 'http://' + hostnameRef.current.value
+
       const payload = {
         overseerr_url: addPortToUrl(hostnameVal, +portRef.current.value),
         overseerr_api_key: apiKeyRef.current.value,

diff --git a/ui/src/components/Settings/Radarr/index.tsx b/ui/src/components/Settings/Radarr/index.tsx
@@ -74,10 +74,12 @@ const RadarrSettings = () => {
       portRef.current?.value &&
       apiKeyRef.current?.value
     ) {
-      const hostnameVal = hostnameRef.current.value.includes('http')
+      const hostnameVal = hostnameRef.current.value.includes('http://')
         ? hostnameRef.current.value
-        : hostnameRef.current.value.includes('https')
+        : hostnameRef.current.value.includes('https://')
           ? hostnameRef.current.value
+          : portRef.current.value == '443' ? 
+          'https://' + hostnameRef.current.value
           : 'http://' + hostnameRef.current.value
 
       let radarr_url = `${addPortToUrl(hostnameVal, +portRef.current.value)}`

diff --git a/ui/src/components/Settings/Sonarr/index.tsx b/ui/src/components/Settings/Sonarr/index.tsx
@@ -74,10 +74,12 @@ const SonarrSettings = () => {
       portRef.current?.value &&
       apiKeyRef.current?.value
     ) {
-      const hostnameVal = hostnameRef.current.value.includes('http')
+      const hostnameVal = hostnameRef.current.value.includes('http://')
         ? hostnameRef.current.value
-        : hostnameRef.current.value.includes('https')
+        : hostnameRef.current.value.includes('https://')
           ? hostnameRef.current.value
+          : portRef.current.value == '443'
+        ? 'https://' + hostnameRef.current.value
           : 'http://' + hostnameRef.current.value
 
       let url = `${addPortToUrl(hostnameVal, +portRef.current.value)}`