X Tutup
Skip to content

Use net.JoinHostPort() to fix formatting with IPv6 addresses#2972

Merged
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:ipv6_port_join
May 25, 2021
Merged

Use net.JoinHostPort() to fix formatting with IPv6 addresses#2972
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:ipv6_port_join

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Feb 15, 2021

Fix formatting with IPv6 addresses

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

@thaJeztah thaJeztah added this to the 21.xx milestone Apr 20, 2021
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter
Copy link

codecov-commenter commented Apr 20, 2021

Codecov Report

Merging #2972 (168173a) into master (daf5f12) will increase coverage by 0.10%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master    #2972      +/-   ##
==========================================
+ Coverage   57.08%   57.18%   +0.10%     
==========================================
  Files         299      299              
  Lines       18689    18689              
==========================================
+ Hits        10668    10688      +20     
+ Misses       7155     7132      -23     
- Partials      866      869       +3     

@thaJeztah
Copy link
Member Author

@silvin-lubecki @AkihiroSuda ptal

1 similar comment
@thaJeztah
Copy link
Member Author

@silvin-lubecki @AkihiroSuda ptal

Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@thaJeztah thaJeztah merged commit 1142740 into docker:master May 25, 2021
@thaJeztah thaJeztah deleted the ipv6_port_join branch May 25, 2021 12:58
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.

4 participants

X Tutup