Skip to content

fix(assets): honor ETag on HEAD requests - #23

Draft
aihuman750 wants to merge 2 commits into
OtterMind:mainfrom
aihuman750:fix/assets-head-etag
Draft

fix(assets): honor ETag on HEAD requests#23
aihuman750 wants to merge 2 commits into
OtterMind:mainfrom
aihuman750:fix/assets-head-etag

Conversation

@aihuman750

Copy link
Copy Markdown
Contributor

What changed

  • Honor If-None-Match on asset HEAD requests.
  • Return 304 Not Modified with the matching ETag instead of an unconditional 200 response.
  • Add a controller regression test for weak ETag matching.

Validation

  • git diff --check
  • Not run locally: Maven and the Maven Wrapper are unavailable in this environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant