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

Multiple content codings are possible #714

Closed
martinthomson opened this issue Jan 29, 2021 · 1 comment · Fixed by #720
Closed

Multiple content codings are possible #714

martinthomson opened this issue Jan 29, 2021 · 1 comment · Fixed by #720

Comments

@martinthomson
Copy link
Contributor

However, this text from item 4 implies otherwise.

If multiple content-codings are acceptable, then the acceptable content-coding with the highest non-zero qvalue is preferred.

I think that this only applies if the server wishes to choose between codings, as opposed to apply them all. Compression codings are usually mutually exclusive, but other codings are not necessarily. Splitting this point into a paragraph might be the way to resolve this:

A message can include multiple content codings [I think no hyphen is correct], but some codings, such as those that compress content, can be redundant. When selecting between multiple redundant content codings, the acceptable content coding with the highest non-zero qvalue is preferred.

@reschke
Copy link
Contributor

reschke commented Jan 31, 2021

See also #719

reschke added a commit that referenced this issue Jan 31, 2021
…levant when some of the acceptable ones are redundant (fixes #714)
reschke added a commit that referenced this issue Jan 31, 2021
Clarify that selection between multiple acceptable codings is only relevant when some of the acceptable ones are redundant (fixes #714)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment