Skip to content
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

Use Host IP address if PrimaryIP is empty #1978

Closed
wants to merge 5 commits into from
Closed

Use Host IP address if PrimaryIP is empty #1978

wants to merge 5 commits into from

Conversation

matq007
Copy link
Contributor

@matq007 matq007 commented Aug 20, 2017

Description

Description

When host network is selected on rancher container, traefik returns as primary IP address "". This results in proxying to http://:PORT. With this PR if the IP address is empty, the returned IP address is host's address where the container is running.

Follow up description: #1498

This is a quick-fix and should be made differently in future. Probably it should start by detecting what network was chosen on a container and based on that return the proper IP address.

@ldez ldez added this to the 1.3 milestone Aug 20, 2017
@ldez ldez requested a review from a team August 20, 2017 22:17
@ldez
Copy link
Member

ldez commented Aug 20, 2017

Thanks for your interest in Traefik ! 👍

You cannot manually edit files in the vendor.

We don't want quick-fix and in respect of this rule I must close your PR.

Rule #1 of open-source: no is temporary, yes is forever.

— Solomon Hykes https://twitter.com/solomonstre/status/715277134978113536

We will be very happy if you open a "non quick-fix" PR on master.

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.

None yet

3 participants