Skip to content
This repository was archived by the owner on Sep 3, 2024. It is now read-only.

Commit 46fdae2

Browse files
etag on metadata
1 parent cdaeb62 commit 46fdae2

3 files changed

Lines changed: 7 additions & 4 deletions

File tree

src/SqlStreamStore.HAL.Tests/StreamMetadataTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ await _fixture.HttpClient.SendAsync(
3030
new HttpRequestMessage(HttpMethod.Get, $"/streams/{StreamId}/metadata")))
3131
{
3232
response.StatusCode.ShouldBe(HttpStatusCode.NotFound);
33+
response.Headers.ETag.ShouldBe(new EntityTagHeaderValue(@"""-1"""));
3334

3435
var resource = await response.AsHal();
3536

@@ -72,6 +73,7 @@ await _fixture.HttpClient.SendAsync(
7273
new HttpRequestMessage(HttpMethod.Get, $"/streams/{StreamId}/metadata")))
7374
{
7475
response.StatusCode.ShouldBe(HttpStatusCode.OK);
76+
response.Headers.ETag.ShouldBe(new EntityTagHeaderValue(@"""0"""));
7577

7678
var resource = await response.AsHal();
7779

src/SqlStreamStore.HAL/Resources/GetStreamMetadataOperation.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ public GetStreamMetadataOperation(HttpRequest request)
1515
public string StreamId { get; }
1616

1717
public Task<StreamMetadataResult> Invoke(IStreamStore streamStore, CancellationToken ct)
18-
{
19-
return streamStore.GetStreamMetadata(StreamId, ct);
20-
}
18+
=> streamStore.GetStreamMetadata(StreamId, ct);
2119
}
2220
}

src/SqlStreamStore.HAL/Resources/StreamMetadataResource.cs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,10 @@ public async Task<Response> Get(
4646
.AddEmbeddedResource(
4747
Constants.Relations.Metadata,
4848
Schemas.SetStreamMetadata),
49-
result.MetadataStreamVersion >= 0 ? 200 : 404);
49+
result.MetadataStreamVersion >= 0 ? 200 : 404)
50+
{
51+
Headers = { [Constants.Headers.ETag] = new[] { $@"""{result.MetadataStreamVersion}""" } }
52+
};
5053

5154
return response;
5255
}

0 commit comments

Comments
 (0)