Skip to content

feat(134): replace etag guidance with 154 reference#356

Merged
toumorokoshi merged 1 commit intoaep-dev:mainfrom
toumorokoshi:yft/fix-241
Sep 30, 2025
Merged

feat(134): replace etag guidance with 154 reference#356
toumorokoshi merged 1 commit intoaep-dev:mainfrom
toumorokoshi:yft/fix-241

Conversation

@toumorokoshi
Copy link
Copy Markdown
Member

@toumorokoshi toumorokoshi commented Sep 29, 2025

Previously, there was bespoke guidance in the Update AEP that refered to using a custom ETag field in the resource for validating previous resource state.

This guidance somewhat duplicates the guidance in RFC 9110 and AEP-154 which states that it should be adhered to. This duplication can lead to partial support of both, which in turn will make client integration more difficult.

Therefore, removing this guidance. Fixes #241

🍱 Types of changes

What types of changes does your code introduce to AEP? Put an x in the boxes
that apply

  • Enhancement
  • New proposal
  • Migrated from google.aip.dev
  • Chore / Quick Fix

📋 Your checklist for this pull request

Please review the AEP Style and Guidance for
contributing to this repository.

General

💝 Thank you!

@toumorokoshi toumorokoshi requested a review from a team as a code owner September 29, 2025 04:59
@toumorokoshi toumorokoshi self-assigned this Sep 29, 2025
@toumorokoshi toumorokoshi moved this to In Review in aep-2026 Sep 29, 2025
Previously, there was bespoke guidance in the Update AEP that
refered to using a custom ETag field in the resource for
validating previous resource state.

This guidance somewhat duplicates the guidance in RFC 9110 
and AEP-154 which states that it should be adhered to. This 
duplication can lead to partial support of both, which in turn
will make client integration more difficult.

Therefore, removing this guidance. Fixes aep-dev#241
@toumorokoshi toumorokoshi merged commit 88bbe47 into aep-dev:main Sep 30, 2025
3 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in aep-2026 Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

replacing usage of update etag with AIP-154 ifMatch?

2 participants