Skip to content
New issue

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

Add the proper Exception message for all JSON error types by PHP #110

Merged
merged 3 commits into from Jun 21, 2017

Conversation

@gbalduzzi
Copy link

gbalduzzi commented Sep 17, 2016

Related to issue #99 Unknown JSON error

json_last_error_msg() is not used because it would break compatibility with PHP < 5.5.
This solution supports all error types until 5.5, keeping compatibility with previous versions.

@googlebot googlebot added the cla: yes label Sep 17, 2016
@bshaffer bshaffer merged commit b2a5316 into firebase:master Jun 21, 2017
2 checks passed
2 checks passed
cla/google All necessary CLAs are signed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@bshaffer bshaffer mentioned this pull request Jun 21, 2017
@T3RON

This comment has been minimized.

Copy link

T3RON commented Jun 6, 2019

**> please send me JWT.php fixed for CodeIgniter ...!

seif4847@gmail.com**

my error :

An uncaught Exception was encountered

Type: DomainException

Message: Unknown JSON error: 5

Filename: F:\wamp64\www\fzm\application\helpers\jwt_helper.php

Line Number: 201

Backtrace:

File: F:\wamp64\www\fzm\application\helpers\jwt_helper.php
Line: 132
Function: _handleJsonError

File: F:\wamp64\www\fzm\application\helpers\jwt_helper.php
Line: 42
Function: jsonDecode

File: F:\wamp64\www\fzm\application\helpers\authorization_helper.php
Line: 18
Function: decode

File: F:\wamp64\www\fzm\application\controllers\api\ApiPost.php
Line: 78
Function: validateToken

File: F:\wamp64\www\fzm\application\libraries\REST_Controller.php
Line: 785
Function: call_user_func_array

File: F:\wamp64\www\fzm\index.php
Line: 315
Function: require_once
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.