Fixed: a duplicate trailing slash was rendered in Location header when request path ended with a slash. For example: /workItems/ => /workItems//1

Author: bkoelman

src/JsonApiDotNetCore/Controllers/BaseJsonApiController.cs

@@ -207,7 +207,7 @@ public virtual async Task<IActionResult> PostAsync([FromBody] TResource resource
         TResource? newResource = await _create.CreateAsync(resource, cancellationToken);
 
         string resourceId = (newResource ?? resource).StringId!;
-        string locationUrl = $"{HttpContext.Request.Path}/{resourceId}";
+        string locationUrl = HttpContext.Request.Path.Add($"/{resourceId}");
 
         if (newResource == null)
         {

test/JsonApiDotNetCoreTests/IntegrationTests/ReadWrite/Creating/CreateResourceTests.cs

@@ -49,7 +49,7 @@ public async Task Sets_location_header_for_created_resource()
             }
         };
 
-        const string route = "/workItems";
+        const string route = "/workItems/";
 
         // Act
         (HttpResponseMessage httpResponse, Document responseDocument) = await _testContext.ExecutePostAsync<Document>(route, requestBody);
@@ -61,7 +61,7 @@ public async Task Sets_location_header_for_created_resource()
         httpResponse.Headers.Location.Should().Be($"/workItems/{newWorkItemId}");
 
         responseDocument.Links.ShouldNotBeNull();
-        responseDocument.Links.Self.Should().Be("http://localhost/workItems");
+        responseDocument.Links.Self.Should().Be("http://localhost/workItems/");
         responseDocument.Links.First.Should().BeNull();
 
         responseDocument.Data.SingleValue.ShouldNotBeNull();