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

add -dynamic-destination flag #5

Merged
merged 3 commits into from
Mar 24, 2024
Merged

add -dynamic-destination flag #5

merged 3 commits into from
Mar 24, 2024

Conversation

kzemek
Copy link
Owner

@kzemek kzemek commented Mar 24, 2024

Hi!

We've a setup where multiple services should be available in the destination host. This flag allow the client to select the desired port in the PROXY protocol header.

Of course, the client should be trusted for this to work without security issues (in our case, its our frontal TCP reverse-proxy).

Any comment is welcome, and thanks for this helpful little tool! ;)

By @Xfennec at path-network/go-mmproxy#15

@kzemek kzemek added the feature This PR contains a non-breaking feature label Mar 24, 2024
@kzemek kzemek marked this pull request as ready for review March 24, 2024 15:32
@kzemek kzemek merged commit 2e7b7ca into main Mar 24, 2024
3 checks passed
@kzemek kzemek deleted the dynamic-destination branch March 24, 2024 15:32
@Xfennec
Copy link

Xfennec commented Mar 25, 2024

Thanks for the merge!

It may not be the right place for that, but can you explain the difference between path-network/go-mmproxy and kzemek/go-mmproxy repositories?

@kzemek
Copy link
Owner Author

kzemek commented Mar 25, 2024

@Xfennec sure! kzemek/go-mmproxy is my personal repository where I'll continue maintaining & developing go-mmproxy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This PR contains a non-breaking feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants