IBX-12046: Removed Symfony 8 deprecated code usage

[ViniTou]

🎫 Issue	IBX-12046

Related PRs:

• [TMP] IBX-12046: Removed Symfony 8 deprecated code usage core#822

Description:

Symfony 8.0 removes Request::get(). This repo had one direct call to it in
ConditionallyRemoveVaryHeaderListener::onKernelResponse(), reading the
current route name (_route) off the request to decide whether to strip the
Vary header. _route is a route attribute, so the fix replaces
$event->getRequest()->get('_route') with
$event->getRequest()->attributes->get('_route').

The unit test mocked Request::get() directly, which silently hid the
deprecation (mocks never execute the real, deprecated method). It now
constructs a real Request and sets the _route attribute on it, so the
test actually exercises the code path.

No BC-breaking constraint changes were needed: the other static-scan hit
(TagHandler::__construct() calling parent::__construct($options)) extends
FOS's SymfonyResponseTagger (which extends FOS\HttpCache\ResponseTagger),
not a Symfony Constraint subclass, so the #[HasNamedArguments] recipe does
not apply there.

For QA:

Exercise any route configured under ibexa_http_cache.remove_default_vary_cookie_headers
(or equivalent config consuming ConditionallyRemoveVaryHeaderListener) and
confirm the Vary header is still stripped of Cookie/Authorization for
matched routes, and left untouched for other routes.

Documentation:

N/A

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud