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
mime: encode CTL and non-US-ASCII characters in FormatMediaType #29328
Conversation
This PR (HEAD: 9c77146) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/154760 to see it. Tip: You can toggle comments from me using the |
Message from Bryan C. Mills: Patch Set 1: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
Message from Gobot Gobot: Patch Set 1: TryBots beginning. Status page: https://farmer.golang.org/try?commit=ba5c7e0c Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
Message from Gobot Gobot: Patch Set 1: Build is still in progress... Consult https://build.golang.org/ to see whether it's a new failure. Other builds still in progress; subsequent failure notices suppressed until final report. Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
Message from Brad Fitzpatrick: Patch Set 2: Patch Set 1 was rebased Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
Message from Brad Fitzpatrick: Patch Set 2: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
Message from Gobot Gobot: Patch Set 2: TryBots beginning. Status page: https://farmer.golang.org/try?commit=3ad1054f Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
Message from Gobot Gobot: Patch Set 2: TryBot-Result+1 TryBots are happy. Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
Message from Agniva De Sarker: Patch Set 2: Brad ? Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
Encodes non-WSP CTL and non-US-ASCII UTF-8 characters using syntax specified in RFC 2231. Fixes #7668 Fixes #9624 Change-Id: I433f167c5bdd84a7f811ac0410b08b10790e0d9f GitHub-Last-Rev: 9c77146 GitHub-Pull-Request: #29328 Reviewed-on: https://go-review.googlesource.com/c/go/+/154760 Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Message from Brad Fitzpatrick: Patch Set 3: Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/154760. |
This PR is being closed because golang.org/cl/154760 has been merged. |
Naive question: How can I tell what version of Go this will be / is released in? |
1.13. |
Encodes non-WSP CTL and non-US-ASCII UTF-8 characters using syntax specified in RFC 2231.
Fixes #7668
Fixes #9624