Updated code review

Author: haphan

src/Common/Service/Builder.php

@@ -139,14 +139,14 @@ private function getStack(callable $authHandler, Token $token = null): HandlerSt
         return $stack;
     }
 
-    private function httpClient(string $baseUrl, HandlerStack $stack, string $serviceType, string $microVersion = null): ClientInterface
+    private function httpClient(string $baseUrl, HandlerStack $stack, string $serviceType = null, string $microVersion = null): ClientInterface
     {
         $clientOptions = [
             'base_uri' => Utils::normalizeUrl($baseUrl),
             'handler'  => $stack,
         ];
 
-        if ($microVersion) {
+        if ($microVersion && $serviceType) {
             $clientOptions['headers']['OpenStack-API-Version'] = sprintf('%s %s', $serviceType, $microVersion);
         }
 

src/Compute/v2/Params.php

@@ -456,7 +456,8 @@ public function userId(): array
         return [
             'type'     => self::STRING_TYPE,
             'sentAs'   => 'user_id',
-            'location' => self::QUERY
+            'location' => self::QUERY,
+            'description' => 'This allows administrative users to operate key-pairs of specified user ID. Requires micro version 2.10.'
         ];
     }