refs kbsali#2 allow to turn on ssl certificate check and to set the port number

Author: kbsali

README.markdown

@@ -74,7 +74,7 @@ $client->api('issue')->create(array(
 
 see `example.php`
 
-### Contributors
+### Thanks!
 
 - Thanks to [Thomas Spycher](http://tspycher.com/2011/03/using-the-redmine-api-with-php/) for the 1st version of the class.
 - Thanks to [Thibault Duplessis aka. ornicar](https://github.com/ornicar) for the php-github-api library, great source of inspiration!

example.php

@@ -4,6 +4,16 @@
 
 $client = new Redmine\Client('http://redmine.example.com', '1234567890abcdfgh');
 
+// ----------------------------
+// [OPTIONAL] if you want to check
+// the servers' SSL certificate on Curl call
+$client->setCheckSslCertificate(true);
+
+// ----------------------------
+// [OPTIONAL] set the port
+// (it will try to guess it from the url)
+$client->setPort(8080);
+
 // ----------------------------
 // Trackers
 $client->api('tracker')->all();

lib/Redmine/Client.php

@@ -17,6 +17,11 @@ class Client
         'https' => 443,
     );
 
+    /**
+     * @var int
+     */
+    private $port;
+
     /**
      * @var string
      */
@@ -27,6 +32,11 @@ class Client
      */
     private $apikey;
 
+    /**
+     * @var boolean
+     */
+    private $checkSslCertificate = false;
+
     /**
      * @param string $url
      * @param string $apikey
@@ -166,6 +176,53 @@ public function delete($path)
         return $this->runRequest($path, 'DELETE');
     }
 
+    /**
+     * Turns on/off ssl certificate check
+     * @param boolean $check
+     */
+    public function setCheckSslCertificate($check = false)
+    {
+        $this->checkSslCertificate = $check;
+    }
+
+    /**
+     * Set the port of the connection
+     * @param int $port
+     */
+    public function setPort($port = null)
+    {
+        if (null !== $port) {
+            $this->port = (int) $port;
+        }
+    }
+
+    /**
+     * Returns the port of the current connection,
+     * if not set, it will try to guess the port
+     * from the given $urlPath
+     * @param  string $urlPath the url called
+     * @return int
+     */
+    public function getPort($urlPath = null)
+    {
+        if (null === $urlPath) {
+            return $this->port;
+        }
+        if (null !== $this->port) {
+            return $this->port;
+        }
+        $tmp = parse_url($urlPath);
+
+        if (isset($tmp['port'])) {
+            $this->setPort($tmp['port']);
+
+            return $this->port;
+        }
+        $this->setPort(self::$defaultPorts[$tmp['scheme']]);
+
+        return $this->port;
+    }
+
     /**
      * @param  string                        $path
      * @param  string                        $method
@@ -175,13 +232,7 @@ public function delete($path)
      */
     private function runRequest($path, $method = 'GET', $data = '')
     {
-        $tmp = parse_url($this->url.$path);
-
-        if (isset($tmp['port'])) {
-            $port = $tmp['port'];
-        } else {
-            $port = self::$defaultPorts[$tmp['scheme']];
-        }
+        $this->getPort($this->url.$path);
 
         $curl = curl_init();
         if (isset($this->apikey)) {
@@ -192,11 +243,12 @@ private function runRequest($path, $method = 'GET', $data = '')
         curl_setopt($curl, CURLOPT_VERBOSE, 0);
         curl_setopt($curl, CURLOPT_HEADER, 0);
         curl_setopt($curl, CURLOPT_RETURNTRANSFER, 1);
-        curl_setopt($curl, CURLOPT_PORT , $port);
-        if (80 !== $port) {
-            curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, false);
+        curl_setopt($curl, CURLOPT_PORT , $this->port);
+        if (80 !== $this->port) {
+            curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, $this->checkSslCertificate);
         }
 
+        $tmp = parse_url($this->url.$path);
         if ('xml' === substr($tmp['path'], -3)) {
             curl_setopt($curl, CURLOPT_HTTPHEADER, array(
                 'Content-Type: text/xml',