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

Split http and stream variants #11

Open
jirutka opened this issue Jun 13, 2024 · 0 comments
Open

Split http and stream variants #11

jirutka opened this issue Jun 13, 2024 · 0 comments

Comments

@jirutka
Copy link

jirutka commented Jun 13, 2024

It builds ngx_http_dynamic_upstream_module with ngx_module_type=HTTP, but it actually provides support even for the stream context and hence links the stream module. That’s quite unexpected.

Can you please split the module into http and stream variants? You can take inspiration from https://github.com/kjdev/nginx-keyval or https://github.com/leev/ngx_http_geoip2_module.

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

1 participant