401 responses not correctly error-handled

[dulange]

When the Rocket.Chat REST API responds with 401 due to invalid authentication, the response looks like this (prettified):

{
  "status": "error",
  "message": "You must be logged in to do this."
}

However, the error handling in the Request::send method is a mere success handler (literally, by checking if the JSON response has a success property with a falsy value):

rocket-chat/src/Common/Request.php

Lines 125 to 139 in 66895dc

	static::$response = @json_decode($responseBody);
	static::$responseCode = $responseCode;
	static::$responseUrl = (!empty($headersRedirect)) ? $headersRedirect[count($headersRedirect) - 1] : null;
	if (isset(static::$response->success) && (!static::$response->success)) {
	if (isset(static::$response->error)) {
	static::setError(static::$response->error);
	} else {
	static::setError("Unknown error occurred in api");
	}
	static::$success = false;
	}
	return ((static::$responseCode >= 200) && (static::$responseCode < 300));

The final bool return value of the method, (static::$responseCode >= 200) && (static::$responseCode < 300), could handle this case but Request::sends return value appears to be unused elsewhere in the package.