Skip to content

PHP 8.5#501

Open
W0rma wants to merge 4 commits intojumbojett:masterfrom
W0rma:php85
Open

PHP 8.5#501
W0rma wants to merge 4 commits intojumbojett:masterfrom
W0rma:php85

Conversation

@W0rma
Copy link
Contributor

@W0rma W0rma commented Mar 20, 2026

List of common tasks a pull request require complete

  • Changelog entry is added or the pull request don't alter library's functionality

This PR is based on #500 and adds PHP 8.5 to the build matrix.

@W0rma W0rma mentioned this pull request Mar 20, 2026
}

// Convert token params to string format
$post_params = http_build_query($post_data, null, '&', $this->encType);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is already #414 about this change.

I'm going to rebase if you prefer merging this PR beforehand.

@W0rma
Copy link
Contributor Author

W0rma commented Mar 23, 2026

The failing PHP 7.1 job is unrelated.

Seems like roave/security-advisories blocks the installation of PHPUnit < 8.5.52 becaus of GHSA-vvj3-c3rp-c85p. However, PHPUnit 8.5 requires at least PHP 7.2.

One possible solution could be to drop support for PHP 7.1 in jumbojett/openid-connect-php as well.
@DeepDiver1975 Would that be acceptable?

@W0rma W0rma mentioned this pull request Mar 24, 2026
1 task
@W0rma
Copy link
Contributor Author

W0rma commented Mar 24, 2026

@DeepDiver1975 PR rebased - merge conflicts are gone and the CI is green :)

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.

3 participants