Limit allowed codes for api exceptions

contributte · Oct 2, 2018 · 2da0a7c · 2da0a7c
1 parent bca3184
commit 2da0a7c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/src/Exception/Api/ClientErrorException.php b/src/Exception/Api/ClientErrorException.php
@@ -3,6 +3,7 @@
 namespace Apitte\Core\Exception\Api;
 
 use Apitte\Core\Exception\ApiException;
+use Apitte\Core\Exception\Logical\InvalidArgumentException;
 use Throwable;
 
 /**
@@ -16,6 +17,10 @@ class ClientErrorException extends ApiException
 	 */
 	public function __construct(string $message = '', int $code = 400, ?Throwable $previous = null, $context = null)
 	{
+		if ($code < 400 || $code > 499) {
+			throw new InvalidArgumentException(sprintf('%s code could be only in range from 400 to 499', static::class));
+		}
+
 		parent::__construct($message, $code, $previous, $context);
 	}
 

diff --git a/src/Exception/Api/ServerErrorException.php b/src/Exception/Api/ServerErrorException.php
@@ -3,6 +3,7 @@
 namespace Apitte\Core\Exception\Api;
 
 use Apitte\Core\Exception\ApiException;
+use Apitte\Core\Exception\Logical\InvalidArgumentException;
 use Throwable;
 
 /**
@@ -13,6 +14,10 @@ class ServerErrorException extends ApiException
 
 	public function __construct(string $message = '', int $code = 500, ?Throwable $previous = null)
 	{
+		if ($code < 500 || $code > 599) {
+			throw new InvalidArgumentException(sprintf('%s code could be only in range from 500 to 599', static::class));
+		}
+
 		parent::__construct($message, $code, $previous);
 	}