#3577 moved content negotiation to before firewall

Author: Joe Bennett

src/Bridge/Symfony/Bundle/Resources/config/api.xml

@@ -147,15 +147,15 @@
 
         <!-- Event listeners -->
 
-        <!-- kernel.request priority must be < 8 to be executed after the Firewall -->
         <service id="api_platform.listener.request.add_format" class="ApiPlatform\Core\EventListener\AddFormatListener">
             <argument type="service" id="api_platform.negotiator" />
             <argument type="service" id="api_platform.metadata.resource.metadata_factory" />
             <argument>%api_platform.formats%</argument>
 
-            <tag name="kernel.event_listener" event="kernel.request" method="onKernelRequest" priority="7" />
+            <tag name="kernel.event_listener" event="kernel.request" method="onKernelRequest" priority="28" />
         </service>
 
+        <!-- kernel.request priority must be < 8 to be executed after the Firewall -->
         <service id="api_platform.listener.request.read" class="ApiPlatform\Core\EventListener\ReadListener">
             <argument type="service" id="api_platform.collection_data_provider" />
             <argument type="service" id="api_platform.item_data_provider" />

src/EventListener/AddFormatListener.php

@@ -66,9 +66,11 @@ public function onKernelRequest(RequestEvent $event): void
     {
         $request = $event->getRequest();
         if (
-                !($request->attributes->has('_api_resource_class')
-                || $request->attributes->getBoolean('_api_respond', false)
-                || $request->attributes->getBoolean('_graphql', false))
+                !(
+                    $request->attributes->has('_api_resource_class')
+                    || $request->attributes->getBoolean('_api_respond', false)
+                    || $request->attributes->getBoolean('_graphql', false)
+                )
         ) {
             return;
         }