Nosial/socialbox-php
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Made message signing in Cryptography use SHA512 as the message content for... #1

Closed
netkas wants to merge 421 commits from master into dev
pull from: master
merge into: Nosial:dev
Conversation 3 Commits 421 Files changed 226 +5 -3
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 24530f5494 - Show all commits

5
src/Socialbox/Classes/StandardMethods/Verification/VerificationAnswerImageCaptcha.php
View file

@ -6,6 +6,7 @@
use Socialbox\Enums\Flags\SessionFlags;
use Socialbox\Enums\StandardError;
use Socialbox\Exceptions\DatabaseOperationException;
use Socialbox\Exceptions\Standard\MissingRpcArgumentException;
use Socialbox\Exceptions\Standard\StandardRpcException;
use Socialbox\Interfaces\SerializableInterface;
use Socialbox\Managers\CaptchaManager;
@ -23,7 +24,7 @@
{
if(!$rpcRequest->containsParameter('answer'))
{
return $rpcRequest->produceError(StandardError::RPC_INVALID_ARGUMENTS, 'The answer parameter is required');
throw new MissingRpcArgumentException('answer');
}
$session = $request->getSession();
@ -32,7 +33,7 @@
{
if(CaptchaManager::getCaptcha($session->getPeerUuid())?->isExpired())
{
return $rpcRequest->produceError(StandardError::CAPTCHA_EXPIRED, 'The captcha has expired');
return $rpcRequest->produceError(StandardError::EXPIRED, 'The captcha has expired');
}
}
catch(DatabaseOperationException $e)

3
src/Socialbox/Enums/StandardError.php
View file

@ -16,7 +16,8 @@
case NOT_FOUND = -105;
case RESOLUTION_FAILED = -106;
case CONFLICT = -107;
case CRYPTOGRAPHIC_ERROR = -108;
case EXPIRED = -108;
case CRYPTOGRAPHIC_ERROR = -109;
// RPC Errors
case RPC_METHOD_NOT_FOUND = -1000;