Skip to content

vndr go-connections to v0.4.0 - drop support for TLS < 1.2#37660

Merged
vdemeester merged 1 commit intomoby:masterfrom
andrewhsu:go-con
Aug 17, 2018
Merged

vndr go-connections to v0.4.0 - drop support for TLS < 1.2#37660
vdemeester merged 1 commit intomoby:masterfrom
andrewhsu:go-con

Conversation

@andrewhsu
Copy link
Copy Markdown
Contributor

- What I did

Update vendor of go-connections to latest release v0.4.0. Comparison of upstream changes: docker/go-connections@7beb39f...7395e3f

- How I did it

$ vi vendor.conf
$ make BIND_DIR=. shell
$ vndr github.com/docker/go-connections

- How to verify it

$ make BIND_DIR=. shell
$ vndr github.com/docker/go-connections

- Description for the changelog

  • Update go-connections to v0.4.0

- A picture of a cute animal (not mandatory but encouraged)

🐭

cc @dmcgowan @vdemeester @justincormack

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 17, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@b6242da). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #37660   +/-   ##
=========================================
  Coverage          ?   35.85%           
=========================================
  Files             ?      606           
  Lines             ?    44704           
  Branches          ?        0           
=========================================
  Hits              ?    16027           
  Misses            ?    26469           
  Partials          ?     2208

Copy link
Copy Markdown
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🐯

@vdemeester vdemeester merged commit a7ff19d into moby:master Aug 17, 2018
@andrewhsu andrewhsu deleted the go-con branch August 17, 2018 18:24
@thaJeztah thaJeztah changed the title vndr go-connections to v0.4.0 vndr go-connections to v0.4.0 - drop support for TLS < 1.2 Aug 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants