fix: CURLOPT_SSL_VERIFYPEER values and tests related with CURLOPT_SSL_VERIFYPEER and CURLOPT_SSL_VERIFYHOST

Author: NicolaeIotu

system/HTTP/CURLRequest.php

@@ -544,10 +544,10 @@ protected function setCURLOptions(array $curlOptions = [], array $config = [])
 
                 $curlOptions[CURLOPT_CAINFO] = $file;
                 if ($config['verify'] === 'yes') {
-                    $curlOptions[CURLOPT_SSL_VERIFYPEER] = 1;
+                    $curlOptions[CURLOPT_SSL_VERIFYPEER] = true;
                     $curlOptions[CURLOPT_SSL_VERIFYHOST] = 2;
                 } else {
-                    $curlOptions[CURLOPT_SSL_VERIFYPEER] = 0;
+                    $curlOptions[CURLOPT_SSL_VERIFYPEER] = false;
                     $curlOptions[CURLOPT_SSL_VERIFYHOST] = 0;
                 }
             } elseif (is_bool($config['verify'])) {

tests/system/HTTP/CURLRequestDoNotShareOptionsTest.php

@@ -545,7 +545,7 @@ public function testSSLVerification(): void
         $this->assertSame($file, $options[CURLOPT_CAINFO]);
 
         $this->assertArrayHasKey(CURLOPT_SSL_VERIFYPEER, $options);
-        $this->assertSame(1, $options[CURLOPT_SSL_VERIFYPEER]);
+        $this->assertTrue($options[CURLOPT_SSL_VERIFYPEER]);
     }
 
     public function testSSLWithBadKey(): void

tests/system/HTTP/CURLRequestTest.php

@@ -528,12 +528,27 @@ public function testSSLVerification(): void
         $this->assertSame($file, $options[CURLOPT_CAINFO]);
 
         $this->assertArrayHasKey(CURLOPT_SSL_VERIFYPEER, $options);
-        $this->assertSame(1, $options[CURLOPT_SSL_VERIFYPEER]);
+        $this->assertTrue($options[CURLOPT_SSL_VERIFYPEER]);
 
         $this->assertArrayHasKey(CURLOPT_SSL_VERIFYHOST, $options);
         $this->assertSame(2, $options[CURLOPT_SSL_VERIFYHOST]);
     }
 
+    public function testNoSSL(): void
+    {
+        $this->request->request('get', 'http://example.com', [
+            'verify' => false,
+        ]);
+
+        $options = $this->request->curl_options;
+
+        $this->assertArrayHasKey(CURLOPT_SSL_VERIFYPEER, $options);
+        $this->assertFalse($options[CURLOPT_SSL_VERIFYPEER]);
+
+        $this->assertArrayHasKey(CURLOPT_SSL_VERIFYHOST, $options);
+        $this->assertSame(0, $options[CURLOPT_SSL_VERIFYHOST]);
+    }
+
     public function testSSLWithBadKey(): void
     {
         $file = 'something_obviously_bogus';