Restore "userAgent" MultiHttpClient constructor override
authorAaron Schulz <aschulz@wikimedia.org>
Tue, 6 Oct 2015 22:26:59 +0000 (15:26 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Tue, 6 Oct 2015 22:29:42 +0000 (22:29 +0000)
This is more convenient than needing to set it on each
HTTP request. Also, it is still documented as being there.

Change-Id: I246c9c3861d72701c844200b82d1a38a0c301795

includes/libs/MultiHttpClient.php

index b6aeda5..c6fa914 100644 (file)
@@ -55,6 +55,8 @@ class MultiHttpClient {
        protected $maxConnsPerHost = 50;
        /** @var string|null proxy */
        protected $proxy;
+       /** @var string */
+       protected $userAgent = 'wikimedia/multi-http-client v1.0';
 
        /**
         * @param array $options
@@ -345,7 +347,7 @@ class MultiHttpClient {
                }
 
                if ( !isset( $req['headers']['user-agent'] ) ) {
-                       $req['headers']['user-agent'] = self::userAgent();
+                       $req['headers']['user-agent'] = $this->userAgent;
                }
 
                $headers = array();
@@ -416,12 +418,4 @@ class MultiHttpClient {
                        curl_multi_close( $this->multiHandle );
                }
        }
-
-       /**
-        * The default User-Agent for requests.
-        * @return string
-        */
-       public static function userAgent() {
-               return 'wikimedia/multi-http-client v1.0';
-       }
 }