Skip to content

Commit

Permalink
feat(134): make etag not required
Browse files Browse the repository at this point in the history
making etag a require field will force the user
to perform a Get of the resource first to retrieve the
etag - this can cause friction for simple clients like
command-line interfaces or curl-ing a resource, and 
makes the behavior of the standard method inconsistent
across resources.
  • Loading branch information
toumorokoshi committed Nov 5, 2024
1 parent 1c968b7 commit b055138
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions aep/general/0134/aep.md.j2
Original file line number Diff line number Diff line change
Expand Up @@ -310,17 +310,19 @@ message Book {
}
```

- The `update_mask` field in the request does not affect the behavior of the
`etag` field, as it is not a field _being_ updated.

{% tab oas %}

**Note:** OAS example not yet written.

{% endtabs %}

The `etag` field **may** be either [required][] or [optional][]. If it is set,
then the request **must** succeed if and only if the provided etag matches the
server-computed value, and **must** fail with an `ABORTED` error otherwise. The
`update_mask` field in the request does not affect the behavior of the `etag`
field, as it is not a field _being_ updated.
- The `etag` field **must** be [optional][].
- If `etag` is set, then the request **must** succeed if and only if the
provided etag matches the server-computed value, and **must** fail with an
`ABORTED` / `409 Conflict` error otherwise.

### Expensive fields

Expand Down

0 comments on commit b055138

Please sign in to comment.