ElyBy-Mirror/oauth2-server
1
0
Fork 0
mirror of https://github.com/elyby/oauth2-server.git synced 2024-12-23 13:39:48 +05:30
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'hotfix/auth-exceptions' into develop

Browse Source
This commit is contained in:
Alex Bilbie 2012-09-19 19:53:16 +01:00
commit 602bc85121
2 changed files with 36 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
src/Oauth2/Authentication/Server.php
View File

@ -2,17 +2,17 @@
namespace Oauth2\Authentication;
class OAuthServerClientException extends \Exception
class ClientException extends \Exception
{
}
class OAuthServerUserException extends \Exception
class UserException extends \Exception
{
}
class OAuthServerException extends \Exception
class ServerException extends \Exception
{
}
@ -127,7 +127,7 @@ class Server
// Client ID
if ( ! isset($authParams['client_id']) && ! isset($_GET['client_id'])) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'client_id'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'client_id'), 0);
} else {
@ -138,7 +138,7 @@ class Server
// Redirect URI
if ( ! isset($authParams['redirect_uri']) && ! isset($_GET['redirect_uri'])) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'redirect_uri'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'redirect_uri'), 0);
} else {
@ -151,13 +151,13 @@ class Server
if ($clientDetails === false) {
throw new OAuthServerClientException($this->errors['invalid_client'], 8);
throw new ClientException($this->errors['invalid_client'], 8);
}
// Response type
if ( ! isset($authParams['response_type']) && ! isset($_GET['response_type'])) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'response_type'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'response_type'), 0);
} else {
@ -166,7 +166,7 @@ class Server
// Ensure response type is one that is recognised
if ( ! in_array($params['response_type'], $this->_responseTypes)) {
throw new OAuthServerClientException($this->errors['unsupported_response_type'], 3);
throw new ClientException($this->errors['unsupported_response_type'], 3);
}
}
@ -189,7 +189,7 @@ class Server
if (count($scopes) === 0) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'scope'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'scope'), 0);
}
$params['scopes'] = array();
@ -200,7 +200,7 @@ class Server
if ($scopeDetails === false) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_scope'], $scope), 4);
throw new ClientException(sprintf($this->errors['invalid_scope'], $scope), 4);
}
@ -325,7 +325,7 @@ class Server
if ( ! isset($authParams['grant_type']) && ! isset($_POST['grant_type'])) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'grant_type'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'grant_type'), 0);
} else {
@ -334,7 +334,7 @@ class Server
// Ensure grant type is one that is recognised
if ( ! in_array($params['grant_type'], $this->_grantTypes)) {
throw new OAuthServerClientException($this->errors['unsupported_grant_type'], 7);
throw new ClientException($this->errors['unsupported_grant_type'], 7);
}
}
@ -350,7 +350,7 @@ class Server
case 'password': // Resource owner password credentials grant
case 'client_credentials': // Client credentials grant
default: // Unsupported
throw new OAuthServerException($this->errors['server_error'] . 'Tried to process an unsuppported grant type.', 5);
throw new ServerException($this->errors['server_error'] . 'Tried to process an unsuppported grant type.', 5);
break;
}
}
@ -370,7 +370,7 @@ class Server
// Client ID
if ( ! isset($authParams['client_id']) && ! isset($_POST['client_id'])) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'client_id'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'client_id'), 0);
} else {
@ -381,7 +381,7 @@ class Server
// Client secret
if ( ! isset($authParams['client_secret']) && ! isset($_POST['client_secret'])) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'client_secret'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'client_secret'), 0);
} else {
@ -392,7 +392,7 @@ class Server
// Redirect URI
if ( ! isset($authParams['redirect_uri']) && ! isset($_POST['redirect_uri'])) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'redirect_uri'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'redirect_uri'), 0);
} else {
@ -409,13 +409,13 @@ class Server
if ($clientDetails === false) {
throw new OAuthServerClientException($this->errors['invalid_client'], 8);
throw new ClientException($this->errors['invalid_client'], 8);
}
// The authorization code
if ( ! isset($authParams['code']) && ! isset($_POST['code'])) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_request'], 'code'), 0);
throw new ClientException(sprintf($this->errors['invalid_request'], 'code'), 0);
} else {
@ -433,7 +433,7 @@ class Server
if ( ! $session) {
throw new OAuthServerClientException(sprintf($this->errors['invalid_grant'], 'code'), 9);
throw new ClientException(sprintf($this->errors['invalid_grant'], 'code'), 9);
} else {
@ -500,11 +500,11 @@ class Server
private function _dbCall()
{
if ($this->_db === null) {
throw new OAuthServerException('No registered database abstractor');
throw new ServerException('No registered database abstractor');
}
if ( ! $this->_db instanceof Database) {
throw new OAuthServerException('Registered database abstractor is not an instance of Oauth2\Authentication\Database');
throw new ServerException('Registered database abstractor is not an instance of Oauth2\Authentication\Database');
}
$args = func_get_args();

30
tests/authentication/server_test.php
View File

@ -90,7 +90,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_checkClientAuthoriseParams_missingClientId()
@ -99,7 +99,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_checkClientAuthoriseParams_missingRedirectUri()
@ -110,7 +110,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_checkClientAuthoriseParams_missingResponseType()
@ -122,7 +122,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_checkClientAuthoriseParams_missingScopes()
@ -136,7 +136,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 4
*/
function test_checkClientAuthoriseParams_invalidScopes()
@ -247,7 +247,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_issueAccessToken_missingGrantType()
@ -256,7 +256,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 7
*/
function test_issueAccessToken_unsupportedGrantType()
@ -267,7 +267,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_completeAuthCodeGrant_missingClientId()
@ -280,7 +280,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_completeAuthCodeGrant_missingClientSecret()
@ -295,7 +295,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_completeAuthCodeGrant_missingRedirectUri()
@ -311,7 +311,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 8
*/
function test_completeAuthCodeGrant_invalidClient()
@ -328,7 +328,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 0
*/
function test_completeAuthCodeGrant_missingCode()
@ -345,7 +345,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerClientException
* @expectedException Oauth2\Authentication\ClientException
* @expectedExceptionCode 9
*/
function test_completeAuthCodeGrant_invalidCode()
@ -363,7 +363,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerException
* @expectedException Oauth2\Authentication\ServerException
* @expectedExceptionMessage No registered database abstractor
*/
function test_noRegisteredDatabaseAbstractor()
@ -380,7 +380,7 @@ class Authentication_Server_test extends PHPUnit_Framework_TestCase {
}
/**
* @expectedException Oauth2\Authentication\OAuthServerException
* @expectedException Oauth2\Authentication\ServerException
* @expectedExceptionMessage Registered database abstractor is not an instance of Oauth2\Authentication\Database
*/
function test_invalidRegisteredDatabaseAbstractor()