-
-
Notifications
You must be signed in to change notification settings - Fork 156
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
Format 0 must be utf-8 #33
Comments
Good catch! Would you like to send a PR? |
How would these cases be handled:
Is anyone currently using the first when they intend the second/third options? Should equivalent forms be handled? (I maintain a library to parse these Media type formats, "contenttype") |
Anyway, it's a bug. The spec it's clear that's utf-8, and it is how is parsed by node-coap anyway. Pick what version suits you most of the other two. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs within the next 7 days. Please check if the issue is still relevant in the most current version of the adapter and tell us. Also check that all relevant details, logs and reproduction steps are included and update them if needed. Thank you for your contributions. |
This issue has been automatically closed because of inactivity. Please open a new issue if still relevant and make sure to include all relevant details, logs and reproduction steps. Thank you for your contributions. |
Reopening this issue as the bug is still present. |
Format 0 is defined as
text/plain; charset=utf-8
. By definition, text/* defaults to US-ASCII and causes encoding problems if a Unicode-encoded document is served without this charset parameter - e.g. in CoAP-to-HTTP translators.Instead, the module serves option 0 as merely
text/plain
i.e. US-ASCII.The text was updated successfully, but these errors were encountered: