What do you mean by the "patch notes"? Do you mean the contents of https://golang.org/doc/go1.9? Or do you mean the documentation of LocalAddrContextKey at https://golang.org/pkg/net/http/#pkg-variables ?

sorry, I meant release notes, and yes http.LocalAddrContextKey

Thanks. Not every change is mentioned in the release notes, only the ones that seem important or significant.

CC @tombergan Should this change be in the release notes?

ya I figure, it was a shock to me however to find that I now had access to it

@ianlancetaylor sure! it should be documented and also mentioned in the release notes.
It's important for many web server's for stats or the ability to apply ACL's for services that apply different policies for different networks their clients have access from\to.
For example I have a service which face two different public ip addresses behind nat(each has it's own router) and I want to apply rate limiting only for connections coming on a specific internal ip address without running two different listening ports\services.
(the reverse path route consistency is being handled by iptables)

Here is the documentation for LocalAddrContextKey:

// LocalAddrContextKey is a context key. It can be used in
// HTTP handlers with context.WithValue to access the address
// the local address the connection arrived on.
// The associated value will be of type net.Addr.
LocalAddrContextKey = &contextKey{"local-addr"}

@sabey, that looks correct to me? #18686 fixed a bug where LocalAddrContextKey used the listen address instead of the actual accepted address (where they could differ if the listen address was a wildcard). The fixed behavior seems to match the above comment.

As to whether or not this belongs in the release notes, I don't feel strongly and am fine with adding it. It seems roughly at the same level as the ServeMux change called out in the release notes:
https://golang.org/doc/go1.9#net/http

Change https://golang.org/cl/59530 mentions this issue: doc/1.9: add mention of net/http.LocalAddrContextKey

@tombergan looks correct

Change https://golang.org/cl/59670 mentions this issue: [release-branch.go1.9] doc/1.9: add mention of net/http.LocalAddrContextKey

Change https://golang.org/cl/68232 mentions this issue: [release-branch.go1.9] doc/1.9: add mention of net/http.LocalAddrContextKey