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

Broken compatibility with Symfony 4.4 in 2.18 #1131

Open
jakubtobiasz opened this issue Mar 23, 2023 · 4 comments
Open

Broken compatibility with Symfony 4.4 in 2.18 #1131

jakubtobiasz opened this issue Mar 23, 2023 · 4 comments

Comments

@jakubtobiasz
Copy link

Hello,
today, I've noticed 2.18 breaks compatibility with Symfony 4.4 by introducing ApiPlatformOpenApiPass. It fails because on Symfony 4.4 there's no security.firewalls parameter, so the whole pass is skipped. Inside it sets some arguments on the lexik_jwt_authentication.api_platform.openapi.factory service. When this compiler pass is skipped, the 3 of 4 arguments are null in this service definition. It results in the following error:
CleanShot 2023-03-23 at 09 00 44@2x

I see three options:
a) Check if security.firewalls parameter exists, if not set it
b) Drop support for Symfony 4.4 (bad idea as it's a minor version)
c) Remove this service in the pass when the parameter doesn't exist

@chalasr
Copy link
Collaborator

chalasr commented Jul 4, 2023

Hi, apologies for the super late reply.
Option c) looks sensible to me. Would you mind sending a PR?

@jakubtobiasz
Copy link
Author

I'll try to find some spare time to fix this :).

@chalasr
Copy link
Collaborator

chalasr commented Jul 4, 2023

Cool :) Feel free to ping me on the symfony-devs slack anytime

@chalasr
Copy link
Collaborator

chalasr commented Dec 2, 2023

Friendly ping

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

No branches or pull requests

2 participants