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

Jump to bottom

Nginx rpm install doesn't support chunkin module #38

Closed
ChrisLundquist opened this issue Nov 11, 2012 · 4 comments
Closed

Nginx rpm install doesn't support chunkin module #38

ChrisLundquist opened this issue Nov 11, 2012 · 4 comments

Comments

@ChrisLundquist
Copy link

This isn't really your issue, but I wanted to let you know and get your thoughts.

Here is where I am at.

I setup a node with this cookbook, and I setup nginx as a proxy to get basic auth.

My use case is that I want Hubot to push chat logs into elastic search
https://gist.github.com/4050748 # Work In Progress

The issue is that node seems to like streaming requests and doesn't set the Content-Length header.
This makes Nginx do the following

<head><title>411 Length Required</title></head>
<body bgcolor="white">
<center><h1>411 Length Required</h1></center>
<hr><center>nginx/1.0.15</center>
</body>
</html>

It seems like this would be a fairly common use case, which is why I am bringing this to you.

The work around

Currently, I am hand coding the json into a string, and setting the Content-Length header, which is ugly but working.

Alternatives?

http://www.lamnk.com/blog/computer/fix-nginx-411-length-required-error/

suggests compiling with the chunkin module, but then that goes back to community nginx. It was cleaned up a bit a few days ago, but it is still a pretty heavy dependency.

What are your thoughts or suggestions?

Thank You
Chris Lundquist
@karmi
Copy link
Contributor

karmi commented Dec 12, 2012

Chris, sorry it took me so long to answer. From the description, it sounds tricky, as it really means a custom Nginx installation. I think we should support people who want to install their own Nginx by whatever means, and then just provide the proxy config: that would mean splitting the attribute/recipe in two -- sounds something doable and worth doing.

@karmi
Copy link
Contributor

karmi commented Jan 15, 2013

Hi, this has slipped me -- the current cookbook version has the nginx and proxy recipes splitted, so you should be able to use whatever Nginx installation method now. Could you please verify and close the issue if you'll solve it?

@ChrisLundquist
Copy link
Author

Thank you! The quality of work here is excellent.

@karmi
Copy link
Contributor

karmi commented Jan 15, 2013

Thanks!

This was referenced Apr 21, 2013
Closed
Open
saschagoebel added a commit to saschagoebel/elasticsearch-js that referenced this issue Nov 6, 2015
@saschagoebel
Always send Content-Length header. Fixes compatibility issues with el…
6f7e165 
…asticsearch behind nginx.

See sous-chefs/elasticsearch#38 or http://serverfault.com/questions/164220/is-there-a-way-to-avoid-nginx-411-content-length-required-errors
@saschagoebel saschagoebel mentioned this issue Nov 6, 2015
Merged
spalger pushed a commit to elastic/elasticsearch-js that referenced this issue Nov 30, 2015
@saschagoebel
Always send Content-Length header. Fixes compatibility issues with el…
469f901 
…asticsearch behind nginx. See sous-chefs/elasticsearch#38 or http://serverfault.com/questions/164220/is-there-a-way-to-avoid-nginx-411-content-length-required-errors
superdev728 added a commit to superdev728/elasticsearch-js that referenced this issue Jun 1, 2023
@superdev728
Always send Content-Length header. Fixes compatibility issues with el…
41da751 
…asticsearch behind nginx.

See sous-chefs/elasticsearch#38 or http://serverfault.com/questions/164220/is-there-a-way-to-avoid-nginx-411-content-length-required-errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@karmi @ChrisLundquist