Ignore IPv6 addresses in Docker reporter #1552

tomwilkie · 2016-06-04T11:09:05Z

I found this useful why tracking down #1550

probe/docker/container.go

@@ -277,12 +277,15 @@ func (c *container) ports(localAddrs []net.IP) report.StringSet {
 			continue
 		}
 		for _, b := range bindings {
-			if b.HostIP == "0.0.0.0" {
-				for _, ip := range localAddrs {
+			if b.HostIP != "0.0.0.0" {


2opremio · 2016-06-14T16:15:22Z

LGTM

Ignore IPv6 addresses in Docker reporter

3d01622

tomwilkie force-pushed the no-ip-v6 branch from af998b6 to 3d01622 Compare June 14, 2016 08:31

tomwilkie assigned 2opremio Jun 14, 2016

2opremio reviewed Jun 14, 2016
View reviewed changes

tomwilkie merged commit d888509 into master Jun 15, 2016

tomwilkie deleted the no-ip-v6 branch June 15, 2016 15:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ignore IPv6 addresses in Docker reporter #1552

Ignore IPv6 addresses in Docker reporter #1552

tomwilkie commented Jun 4, 2016

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

2opremio commented Jun 14, 2016

Ignore IPv6 addresses in Docker reporter #1552

Ignore IPv6 addresses in Docker reporter #1552

Conversation

tomwilkie commented Jun 4, 2016

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

2opremio commented Jun 14, 2016