search

w3c / web-annotation

Web Annotation Working Group repository, see README for links to specs
https://w3c.github.io/web-annotation/
Other
142 stars 30 forks source link

Fix 333 multiple http headers #338

Closed BigBlueHat closed 8 years ago

BigBlueHat commented 8 years ago

NOTE: this PR "stacks" on the other existing PRs. I've sent. Let me know if/when rebasing is needed.

This fixes #333 by changing the text to match the definition of the include parameter for Prefer which is defined by LDP.

In the end, this actually avoids the multiple Prefer header situation entirely, though it does mean that implementors who supported the multi-Prefer header situation will need to change code to use this space-delimited URI list stored inside a single Prefer header.

Let me know if it needs tweaks. :smile:

BigBlueHat commented 8 years ago

Here's the rawgit link for more sensible reviewing: https://rawgit.com/w3c/web-annotation/fix-333-multiple-http-headers/protocol/wd/index.html#container-representation-preferences

BigBlueHat commented 8 years ago

@iherman @azaroth42 changes added--hopefully in the right place and right order. :smiley: