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

Stop dynamic TCP listener when upstream is no longer available #798

Merged
merged 2 commits into from
Nov 30, 2020

Conversation

fwkz
Copy link
Contributor

@fwkz fwkz commented Oct 30, 2020

This PR closes #796

@CLAassistant
Copy link

CLAassistant commented Oct 30, 2020

CLA assistant check
All committers have signed the CLA.

proxy/serve.go Outdated Show resolved Hide resolved
proxy/serve.go Outdated Show resolved Hide resolved
@nathanejohnson nathanejohnson added this to the 1.5.15 milestone Nov 16, 2020
proxy/serve.go Show resolved Hide resolved
@nathanejohnson
Copy link
Member

Tentatively this looks good, going to pull this down and test it locally today. If all goes well I'll merge this ASAP.

@nathanejohnson
Copy link
Member

@fwkz thank you for your contribution!

@nathanejohnson nathanejohnson merged commit 019ad7a into fabiolb:master Nov 30, 2020
@fwkz
Copy link
Contributor Author

fwkz commented Nov 30, 2020

@fwkz thank you for your contribution!

I'm glad that I could help. 💪

@fwkz fwkz deleted the stop-dynamic-proxy branch November 30, 2020 19:19
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.

TCP Dynamic Proxy is not releasing ports from deregistered services
3 participants