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

Consider switching to pires/go-proxyproto #3

Open
emersion opened this issue Oct 24, 2020 · 1 comment
Open

Consider switching to pires/go-proxyproto #3

emersion opened this issue Oct 24, 2020 · 1 comment

Comments

@emersion
Copy link

Consider switching to pires/go-proxyproto, which has more features than your package.

Don't get me wrong, I don't dislike your package or find it low quality. It's just that pires/go-proxyproto already has a lot more features implemented and has more users (so less potential bugs).

I'd like to add more features to this Caddy plugin (e.g. TLVs). I've already took some time to improve pires/go-proxyproto, I'd prefer to avoid having to re-implement everything in mastercactapus/proxyprotocol. Would you welcome a patch that switches over to pires/go-proxyproto?

Thanks.

@mholt
Copy link

mholt commented Oct 24, 2023

@mastercactapus I just wanted to bring up a discussion here, to see what your thoughts are. PROXY protocol support is ambitious and you've been valiantly providing it for years through your library, which the Caddy project has benefited from greatly. So, thank you for that.

We are starting to get requests for additional features and patches that I imagine would be a burden on your time as a volunteer maintainer.

Could we help relieve some of that burden with a patch to use go-proxyproto? I actually like the simplicity of your package better, and by all means I think you should keep maintaining it if you'd like, I just feel like go-proxyproto might be able to serve more Caddy users with less burden on you. And we'd still proudly use your package (this one) as the Caddy integration.

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

No branches or pull requests

2 participants