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

[Composer] Added conflict with Symfony 5.2.6 #4

Merged
merged 1 commit into from
Mar 30, 2021

Conversation

mnocon
Copy link
Contributor

@mnocon mnocon commented Mar 30, 2021

Question Answer
JIRA issue n/a
Type not sure
Target Ibexa version master (3.3 edition)
BC breaks no
Doc needed no

Symfony 5.2.6 has been released yesterday and it contains a regression:
symfony/symfony#40618

It has been detected by our tests as well:
https://travis-ci.com/github/ezsystems/ezplatform/jobs/494701655
(screenshot: https://res.cloudinary.com/ezplatformtravis/image/upload/v1617084143/screenshots/6062beef4f6f1703756003-vendor_ezsystems_ezplatform-admin-ui_features_personas_subtreeeditor_feature_41_x3qfme.png)

If you want to reproduce it locally you can set up a project (Ibexa OSS/DXP 3.3.x) and configure it to use Symfony Proxy according to https://doc.ibexa.co/en/latest/guide/http_cache/#symfony-reverse-proxy - the same exception is shown.

The issue should be fixed in symfony/symfony#40619 , but it has not been released yet.

Lower versions of our product are not affected, as we limit Symfony version there (https://github.com/ezsystems/ezplatform/blob/3.2/composer.json#L143)

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Checked that target branch is set correctly.
  • Asked for a review (ping @ibexa/engineering).

@mnocon mnocon requested a review from a team March 30, 2021 07:13
@mnocon mnocon requested a review from webhdx March 30, 2021 07:35
@adamwojs adamwojs requested a review from a team March 30, 2021 07:39
@mnocon
Copy link
Contributor Author

mnocon commented Mar 30, 2021

@lserwatka can we merge this one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants