Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accept-Query scope #1913

Closed
mnot opened this issue Feb 2, 2022 · 2 comments · Fixed by #2653
Closed

Accept-Query scope #1913

mnot opened this issue Feb 2, 2022 · 2 comments · Fixed by #2653
Assignees

Comments

@mnot
Copy link
Member

mnot commented Feb 2, 2022

Right now, the spec says:

The "Accept-Query" response header field MAY be used by a server to directly signal support for the QUERY method while identifying the specific query format media type(s) that may be used.

The use of 'server' here implies that if a client gets a response from e.g., https://example.com/foo that contains Accept-Query, that client can infer that all other resources on the server support the listed media type(s).

That seems like it's overstating things; other resources on the server might accept other formats, or not accept QUERY at all (indeed, most may not). Given how disruptive such a change would be (e.g., to caching), it seems like a more cautious approach is warranted.

So, I suggest specifying 'resource', not 'server', and explaining the implications in some more detail.

I'm aware that Accept-Patch and Accept-Post both say 'server', but I think that's more from a history of imprecision. Also, neither of those can be read to imply that the specified method is preferred in place of another (especially one as widely used as GET).

@reschke reschke self-assigned this Feb 2, 2022
@mnot
Copy link
Member Author

mnot commented Oct 13, 2022

(I wonder if it would be good to raise technical errata on accept-patch and accept-post about this)

mnot added a commit that referenced this issue Oct 13, 2023
reschke pushed a commit that referenced this issue Nov 5, 2023
@reschke
Copy link
Contributor

reschke commented Nov 6, 2023

(I wonder if it would be good to raise technical errata on accept-patch and accept-post about this)

Accept-Patch: yes.

Accept-Post: IANA points to https://www.w3.org/TR/ldp/ which mentions the expired ID https://datatracker.ietf.org/doc/draft-wilde-accept-post/ - maybe this should be resurrected. As POST is defined in the core specs, this could go there in a future revision, too (maybe open a reminder ticket over there?)

pmeenan pushed a commit to pmeenan/http-extensions that referenced this issue Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants