wip

nextcloud · Jan 8, 2025 · b7b61eb · b7b61eb
1 parent df01931
commit b7b61eb
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 1 deletion.
diff --git a/lib/Config/FetcherConfig.php b/lib/Config/FetcherConfig.php
@@ -129,7 +129,7 @@ public function __construct(
 
         $this->logger->debug(
             'Proxy configuration finalized: {proxy}',
-            ['proxy' => $proxy]
+            ['proxy' => $this->proxy]
         );
 
         return $this;
@@ -198,4 +198,14 @@ public function getUserAgent(): string
 
         return 'NextCloud-News/' . $this->version;
     }
+
+    /**
+     * Get the proxy configuration
+     *
+     * @return string|null
+     */
+    public function getProxy(): ?string
+    {
+        return $this->proxy;
+    }
 }
diff --git a/tests/Unit/Config/FetcherConfigTest.php b/tests/Unit/Config/FetcherConfigTest.php
@@ -122,4 +122,29 @@ public function testGetUserAgentUnknownVersion()
         $response = $this->class->getUserAgent();
         $this->assertEquals($expected, $response);
     }
+
+    public function testProxyPortPreserved()
+    {
+        $this->config->expects($this->exactly(2))
+                     ->method('getValueInt')
+                     ->willReturnMap([
+                        ['news', 'feedFetcherTimeout', 60, false, 60],
+                        ['news', 'maxRedirects', 10, false, 10]
+                     ]);
+
+        $this->appmanager->expects($this->exactly(1))
+                         ->method('getAppVersion')
+                         ->willReturn('1.0');
+
+        $this->sysconfig->expects($this->exactly(1))
+                        ->method('getSystemValue')
+                        ->with('proxy')
+                        ->willReturn('http://192.168.178.1:80');
+
+        $this->class = new FetcherConfig($this->config, $this->sysconfig, $this->appmanager, $this->logger);
+
+        $expected = 'http://192.168.178.1:80';
+        $response = $this->class->getProxy();
+        $this->assertEquals($expected, $response);
+        }
 }