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

325 byte order mark in text/plain #2017

Closed
wants to merge 8 commits into from

Conversation

ralfhandl
Copy link
Contributor

Fixes #325

Copy link

@Hubert-Heijkers Hubert-Heijkers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes are fine but, unfortunately, only apply to text/plain (which in my mind never were an issue as that's US-ASCII anyway, to being explicit is a good clarification).

@Hubert-Heijkers
Copy link

I realize the title of this issue being resolved says "use of byte-order mark (BOM) in text/plain responses" but, as I've expressed originally in a comment on the Jira issue, my interest in this topic of BOMs was more related to responses that actually use a Unicode charset, specifically UTF-8, and as such I'd hoped that we could also make explicit that, based on the remarks wrt the Byte-Order Mark in RFC 3629 that OData services MUST NOT (SHOULD NOT if that would break existing rules) send a byte-order mark in UTF-8 responses (read: irrespective of content-type, with UTF-8 charset implied or explicitly specified using charset parameter).

Should I create a separate issue or can we entertain fixing that as part of this issue as well?

@ralfhandl ralfhandl closed this Nov 13, 2024
@ralfhandl ralfhandl deleted the 325-byte-order-mark-in-text/plain branch November 13, 2024 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarify use of byte-order mark (BOM) in text/plain responses
3 participants