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
go.blog/context: Context blog post Gorilla example has a subtle bug #8443
Labels
Comments
I seem to recall this was originally an int? Owner changed to @Sajmani. Status changed to Accepted. |
Yes, I changed it to use the address of an empty struct var when I updated the example to use just a single key. The example server still works because there's only a single context key used in the example, but this is a bug. I will send a CL to fix the affected examples in the blog post (gorilla.go and userip.go). Note that the context godoc uses an int key and so is ok. |
CL https://golang.org/cl/121890043 mentions this issue. |
This issue was closed by revision golang/blog@9cab33c. Status changed to Fixed. |
minux
pushed a commit
to Go-zh/blog
that referenced
this issue
Dec 14, 2014
key, as this is not guaranteed to be unique. Fixes golang/go#8443. LGTM=campoy R=adg, campoy CC=dsymonds, golang-codereviews https://golang.org/cl/121890043
gopherbot
pushed a commit
to golang/website
that referenced
this issue
May 26, 2021
… a context key, as this is not guaranteed to be unique. Fixes golang/go#8443. LGTM=campoy R=adg, campoy CC=dsymonds, golang-codereviews https://golang.org/cl/121890043 X-Blog-Commit: 9cab33ca8e480d9a24efdd84433601a80f9c7fb5
passionSeven
added a commit
to passionSeven/website
that referenced
this issue
Oct 18, 2022
… a context key, as this is not guaranteed to be unique. Fixes golang/go#8443. LGTM=campoy R=adg, campoy CC=dsymonds, golang-codereviews https://golang.org/cl/121890043 X-Blog-Commit: 9cab33ca8e480d9a24efdd84433601a80f9c7fb5
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The text was updated successfully, but these errors were encountered: