We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error handling api docs says that every error response contains these data:
{ "errorType" : "security", "errorCode" : "901", "message" : "Invalid Merchant Account", "status" : "403" }
But in library method that handle this kind of errors (handleResultError in CurlClient) there are forwarded ONLY message and errorCode.
handleResultError
CurlClient
if(isset($decodeResult['message']) && isset($decodeResult['errorCode'])) { $logger->error($decodeResult['errorCode'] . ': ' . $decodeResult['message']); throw new \Adyen\AdyenException($decodeResult['message'], $decodeResult['errorCode']); }
It would be a lot easier to handle errors if exceptions throws by "adyen-php-api-library" contains also status field.
Regards Przemek
The text was updated successfully, but these errors were encountered:
#29 Add status and ErrorType into the AdyenException
e7b42bf
Hi @przemeko,
Thanks for you comment. Yes this makes sense, see the change above. Is this change the solution that you were looking for?
Regards, Rik Adyen
Sorry, something went wrong.
Hi @rikterbeek
Yes, it solves issue. Great job!
No branches or pull requests
Error handling api docs says that every error response contains these data:
But in library method that handle this kind of errors (
handleResultError
inCurlClient
) there are forwarded ONLY message and errorCode.It would be a lot easier to handle errors if exceptions throws by "adyen-php-api-library" contains also status field.
Regards
Przemek
The text was updated successfully, but these errors were encountered: