Skip to content

IBX-8471: Fixed Varnish::queueRequest implementation after SF 7 upgrade #68

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

Merged
merged 2 commits into from
May 14, 2025

Conversation

alongosz
Copy link
Member

@alongosz alongosz commented May 12, 2025

Caution

  • Drop TMP commits before merging
🎫 Issue IBX-8471

Related PRs:

Description:

Passed missing $validateHost to parent \FOS\HttpCache\ProxyClient\Varnish in \Ibexa\HttpCache\ProxyClient\Varnish::queueRequest implementation.

For QA:

Passing regression tests.

@alongosz alongosz force-pushed the ibx-8471-post-merge-fixes branch from 3ef6e18 to 9f1df3c Compare May 12, 2025 13:53
Copy link
Contributor

@ViniTou ViniTou left a comment

Choose a reason for hiding this comment

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

We may never know how this was working before.
remember about dropping TMP things.

@alongosz alongosz force-pushed the ibx-8471-post-merge-fixes branch 2 times, most recently from 6ff95f7 to f61de54 Compare May 12, 2025 20:25
@alongosz alongosz force-pushed the ibx-8471-post-merge-fixes branch from f61de54 to 5e1a6b1 Compare May 13, 2025 10:33
Interfaces are already announced in '\FOS\HttpCache\ProxyClient\Varnish'
@alongosz alongosz merged commit af853fc into main May 14, 2025
42 of 49 checks passed
@alongosz alongosz deleted the ibx-8471-post-merge-fixes branch May 14, 2025 11:12
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.

4 participants