Merge pull request zendframework/zendframework#5933 from gkralik/fix/json-rpc-error-codes

Allow arbitrary error codes in JSON RPC server

Author: weierophinney

src/Server/Error.php

@@ -18,24 +18,11 @@ class Error
     const ERROR_INTERNAL        = -32603;
     const ERROR_OTHER           = -32000;
 
-    /**
-     * Allowed error codes
-     * @var array
-     */
-    protected $allowedCodes = array(
-        self::ERROR_PARSE,
-        self::ERROR_INVALID_REQUEST,
-        self::ERROR_INVALID_METHOD,
-        self::ERROR_INVALID_PARAMS,
-        self::ERROR_INTERNAL,
-        self::ERROR_OTHER,
-    );
-
     /**
      * Current code
      * @var int
      */
-    protected $code = -32000;
+    protected $code = self::ERROR_OTHER;
 
     /**
      * Error data
@@ -56,11 +43,10 @@ class Error
      * @param  int $code
      * @param  mixed $data
      */
-    public function __construct($message = null, $code = -32000, $data = null)
-    {
+    public function __construct($message = null, $code = self::ERROR_OTHER, $data = null) {
         $this->setMessage($message)
-             ->setCode($code)
-             ->setData($data);
+              ->setCode($code)
+              ->setData($data);
     }
 
     /**
@@ -71,17 +57,17 @@ public function __construct($message = null, $code = -32000, $data = null)
      */
     public function setCode($code)
     {
-        if (!is_scalar($code)) {
+        if (!is_scalar($code) || is_bool($code) || is_float($code)) {
             return $this;
         }
 
-        $code = (int) $code;
-        if (in_array($code, $this->allowedCodes)) {
-            $this->code = $code;
-        } elseif (in_array($code, range(-32099, -32000))) {
-            $this->code = $code;
+        if (is_string($code) && !is_numeric($code)) {
+            return $this;
         }
 
+        $code = (int) $code;
+        $this->code = $code;
+
         return $this;
     }
 

test/Server/ErrorTest.php

@@ -54,7 +54,7 @@ public function testSetCodeShouldCastToInteger()
 
     public function testCodeShouldBeLimitedToStandardIntegers()
     {
-        foreach (array(true, 'foo', array(), new \stdClass, 2.0, 25) as $code) {
+        foreach (array(null, true, 'foo', array(), new \stdClass, 2.0) as $code) {
             $this->error->setCode($code);
             $this->assertEquals(Server\Error::ERROR_OTHER, $this->error->getCode());
         }
@@ -68,6 +68,14 @@ public function testCodeShouldAllowArbitraryAppErrorCodesInXmlRpcErrorCodeRange(
         }
     }
 
+    public function testCodeShouldAllowArbitraryErrorCode()
+    {
+        foreach(array(1000, 404, -3000) as $code) {
+            $this->error->setCode($code);
+            $this->assertEquals($code, $this->error->getCode());
+        }
+    }
+
     public function testMessageShouldBeNullByDefault()
     {
         $this->assertNull($this->error->getMessage());