docker / go-connections

Utility package to work with network connections
https://pkg.go.dev/github.com/docker/go-connections
Apache License 2.0
217 stars 101 forks source link

configureNpipeTransport: use winio.DialPipeContext() #70

Closed thaJeztah closed 4 years ago

thaJeztah commented 4 years ago

This is now in a released version of Microsoft/go-winio

Was going through changes since v0.4.0, and noticed https://github.com/docker/go-connections/pull/58#discussion_r277175135

thaJeztah commented 4 years ago

@SamWhited @AkihiroSuda PTAL

SamWhited commented 4 years ago

If this works, LGTM, but I don't actually see this function in the version of winio in the go.mod file; I must be looking at the wrong one somehow, but I can't for the life of me figure out what I'm doing wrong :S

EDIT: Nevermind, have GOOS set wrong. I should just browse the source instead…