fix(state): skip Content-Location header for GET requests (#6901)

Author: vinceAmstoutz

features/hydra/collection.feature

@@ -77,7 +77,6 @@ Feature: Collections support
     When I send a "GET" request to "/dummies?page=7"
     Then the response status code should be 200
     And the response should be in JSON
-    And the header "Content-Location" should be equal to "/dummies.jsonld?page=7"
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
     And the JSON should be valid according to this schema:
     """

features/main/crud.feature

@@ -60,6 +60,7 @@ Feature: Create-Retrieve-Update-Delete
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -100,13 +101,14 @@ Feature: Create-Retrieve-Update-Delete
   Scenario: Get a not found exception
     When I send a "GET" request to "/dummies/42"
     Then the response status code should be 404
+    And the header "Content-Location" should not exist
 
   Scenario: Get a collection
     When I send a "GET" request to "/dummies"
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
-    And the header "Content-Location" should be equal to "/dummies.jsonld"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -616,6 +618,7 @@ Feature: Create-Retrieve-Update-Delete
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
       {
@@ -640,6 +643,7 @@ Feature: Create-Retrieve-Update-Delete
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
       {
@@ -657,6 +661,7 @@ Feature: Create-Retrieve-Update-Delete
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -676,6 +681,7 @@ Feature: Create-Retrieve-Update-Delete
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -718,6 +724,7 @@ Feature: Create-Retrieve-Update-Delete
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -737,6 +744,7 @@ Feature: Create-Retrieve-Update-Delete
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {

features/main/crud_abstract.feature

@@ -35,6 +35,7 @@ Feature: Create-Retrieve-Update-Delete on abstract resource
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -52,6 +53,7 @@ Feature: Create-Retrieve-Update-Delete on abstract resource
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be valid according to this schema:
     """
     {

features/main/crud_uri_variables.feature

@@ -96,6 +96,7 @@ Feature: Uri Variables
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -164,6 +165,7 @@ Feature: Uri Variables
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -183,6 +185,7 @@ Feature: Uri Variables
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
+    And the header "Content-Location" should not exist
     And the JSON should be equal to:
     """
     {
@@ -200,3 +203,4 @@ Feature: Uri Variables
     When I add "Content-Type" header equal to "application/ld+json"
     And I send a "GET" request to "/companies/1/employees/2"
     Then the response status code should be 404
+    And the header "Content-Location" should not exist

src/State/Processor/RespondProcessor.php

@@ -129,7 +129,7 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
                     $iri = $this->iriConverter->getIriFromResource($operation->getClass(), UrlGeneratorInterface::ABS_PATH, $operation);
                 }
 
-                if ($iri) {
+                if ($iri && 'GET' !== $method) {
                     $location = \sprintf('%s.%s', $iri, $request->getRequestFormat());
                     if (isset($requestParts['query'])) {
                         $location .= '?'.$requestParts['query'];