Skip to content

Conversation

@weierophinney
Copy link
Contributor

Q A
Documentation no
Bugfix no
BC Break no
New Feature yes
RFC no
QA yes

Description

This patch adds support for PHP 8.2. Additionally, it bumps to laminas-servicemanager 3.11, allowing the ability to migrate from container-interop to PSR-11.

- Adds PHP 8.2
- Drops 7.4
- Bumps deps to versions known to work with PHP 8.2

Signed-off-by: Matthew Weier O'Phinney <[email protected]>
Signed-off-by: Matthew Weier O'Phinney <[email protected]>
Since we've updated to laminas-servicemanager 3.11, we can use the PSR-11 interfaces instead.

Signed-off-by: Matthew Weier O'Phinney <[email protected]>
Signed-off-by: Matthew Weier O'Phinney <[email protected]>
The oauth_authorization_codes table required the code_challenge and code_challenge_method columns.
Additionally, the `Laminas\I18n` module needs to be enabled for the LaminasAuthenticationService tests.

Signed-off-by: Matthew Weier O'Phinney <[email protected]>
Copy link
Member

@gsteel gsteel left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@weierophinney weierophinney merged commit 1936977 into laminas-api-tools:1.9.x Jul 11, 2023
@weierophinney weierophinney deleted the feature/php-8.2-support branch July 11, 2023 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants