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

Breaks Nginx Stream Module #26

Open
RohanNagar opened this issue Oct 9, 2017 · 4 comments
Open

Breaks Nginx Stream Module #26

RohanNagar opened this issue Oct 9, 2017 · 4 comments

Comments

@RohanNagar
Copy link

When I compile this module with the nginx stream module, which is available since 1.9.0, the stream module breaks. Multiple stream tests are failing and resulting in seg faults.

I can try to fix this myself, but if anyone is still active I would appreciate some insight into why you think this might be happening.

I'm on version 1.13.5 of Nginx.

Thanks!

@wandenberg
Copy link
Collaborator

Hi @RohanNagar, I'm trying to fix some known bugs/segfaults in this module.
Can you explain the tests that you are doing? Or even better, provide a vagrant/docker/script where the bug can be easily reproduced?

@RohanNagar
Copy link
Author

@wandenberg I am compiling Nginx 1.13.5 with this module, and the additional configure flags of

--with-stream --with-stream_ssl_module

I'm then running the core nginx tests found here. Almost all of the stream tests are failing.

Honestly, I'm not too sure what these tests are testing, I'm still new to Nginx. However it does seem like there are memory issues. Many times the tests expect a result and instead are getting undefined or empty strings. Also after many of the tests, the error "worker process exited with signal 11" are printed.

I can make a simple script to download these and run them if you would like.

@wandenberg
Copy link
Collaborator

Would be good to have a script.
Have you tried to run the tests without the module?

@RohanNagar
Copy link
Author

Yes, the tests pass without the module. I will put a script together for you soon.

arcivanov added a commit to arcivanov/nginx-upstream-dynamic-servers that referenced this issue Nov 30, 2017
is in the config, the module should not segfault

fixes GUI#26
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