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

Should Charset mention UTF-8 #713

Closed
martinthomson opened this issue Jan 29, 2021 · 6 comments · Fixed by #752
Closed

Should Charset mention UTF-8 #713

martinthomson opened this issue Jan 29, 2021 · 6 comments · Fixed by #752

Comments

@martinthomson
Copy link
Contributor

Accept-Charset is deprecated due to the ubiquity of UTF-8, but the Charset section makes no mention of UTF-8. This might be worth a mention.

@annevk
Copy link
Contributor

annevk commented Jan 29, 2021

It's only used for Accept-Charset though and that section already mentions UTF-8. As does the Media Type section. Since Accept-Charset is deprecated, perhaps the charset production should simply be removed and Accept-Charset takes token directly?

@royfielding
Copy link
Member

Accept-Charset is deprecated primarily for privacy. It also happens to be less useful now because of utf-8. In contrast, the charset parameter is still used most of the time, mainly to reduce sniffing. We can add something about utf-8, but promoting it too hard will cause other (social) problems.

@reschke
Copy link
Contributor

reschke commented Jan 31, 2021

@annevk's observation is correct. But then, charset is used in Content-Type as well, just not referenced directly as it is used over there as a regular parameter.

@annevk
Copy link
Contributor

annevk commented Feb 1, 2021

It's not clear to me at least that the charset production section also applies to Content-Type.

@reschke
Copy link
Contributor

reschke commented Feb 1, 2021

Syntactically, it does not (because that relies on a more generic syntax for arbitrary parameters).

But I agree, this needs cleanup.

@reschke
Copy link
Contributor

reschke commented Feb 7, 2021

Proposal is to remove the ABNF production and to clarify the text (so that it applies to Content-Type as well).

royfielding added a commit that referenced this issue Feb 9, 2021
remove "charset" ABNF production and explain where charsets occur (fixes #713)
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.

5 participants