Skip to content

added ipv6 support #122

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 9, 2015
Merged

added ipv6 support #122

merged 2 commits into from
Oct 9, 2015

Conversation

lukas2511
Copy link
Contributor

These changes add IPv6 support to docker-gen.

IPv6 addresses of docker containers are available as IP6_LinkLocal and IP6_Global next to the old IP string.

I had to update the used docker client library to a newer version with support for container.NetworkSettings.LinkLocalIPv6Address and container.NetworkSettings.GlobalIPv6Address

Proto string
HostIP string
IP string
IP6_LinkLocal string
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you remove the underscores _? I try to keep the code consistent with: https://golang.org/doc/effective_go.html#mixed-caps

@lukas2511
Copy link
Contributor Author

As you wished I removed the underscores from the name of the IPv6 fields

@jwilder jwilder merged commit 92372fe into nginx-proxy:master Oct 9, 2015
@jwilder
Copy link
Collaborator

jwilder commented Oct 9, 2015

Thanks @lukas2511!

@jwilder jwilder mentioned this pull request Oct 9, 2015
punkstar added a commit to punkstar/docker-gen that referenced this pull request Dec 23, 2015
* Document the Network struct introduced by nginx-proxy#144
* Document the IPv6 support introduced by nginx-proxy#122
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants