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

Fixed JWT Bearer Test for PHP 5.3.3 and below #44

Merged
merged 3 commits into from
Jan 24, 2013
Merged

Fixed JWT Bearer Test for PHP 5.3.3 and below #44

merged 3 commits into from
Jan 24, 2013

Conversation

F21
Copy link
Contributor

@F21 F21 commented Jan 23, 2013

PHP 5.3.3 and below does not support openssl on travis-ci.

This pull request should fix it by using the SHA-256 algorithm for generating JWTs for PHP 5.3.3 and below.

@bshaffer
Copy link
Owner

Getting this:

Fatal error: Call to undefined function openssl_sign() in /home/travis/builds/bshaffer/oauth2-server-php/src/OAuth2/Util/JWT.php on line 87

https://travis-ci.org/bshaffer/oauth2-server-php/jobs/4342679

@F21
Copy link
Contributor Author

F21 commented Jan 23, 2013

Yup, that's because PHP 5.2 does not support openssl on travis-ci. I thought that should be fixed with the pull that checked the version (in the last merge) and used a symmetric algorithm for generating the JWT on PHP 5.2, but the build failed.

I am just waiting for this pull to build to see if it passes, because I don't have PHP 5.2 on my VMs.

Build for this pull: https://travis-ci.org/bshaffer/oauth2-server-php/builds/4343292

@F21
Copy link
Contributor Author

F21 commented Jan 23, 2013

Yay, the build passes :)

Must have been an issue with detecting the PHP version.

bshaffer added a commit that referenced this pull request Jan 24, 2013
Fixed JWT Bearer Test for PHP 5.3.3 and below
@bshaffer bshaffer merged commit 6b2f845 into bshaffer:develop Jan 24, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants