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

Upload failing with http2 #132

Open
vlastikcz opened this issue May 18, 2020 · 3 comments
Open

Upload failing with http2 #132

vlastikcz opened this issue May 18, 2020 · 3 comments

Comments

@vlastikcz
Copy link

vlastikcz commented May 18, 2020

Tested environment:

  • CentOS 7
  • Nginx 1.15.9, 1.17.8, 1.18.0
  • upload module 2.3.0, master

Steps:

  • enable ssl and http2 for the upload location
  • try to upload file (with POST and resumable, or with multipart and resumable off)

Result:

  • when resumable on: nginx worker segfault (kernel: nginx[1252]: segfault at 8 ip 000055b16d47b0b5 sp 00007ffe4c711150 error 4 in nginx[55b16d39f000+111000])
  • when resumable off: error *217 premature end of body while sending to client
  • some uploads are usually successful, some fail (I did not see any pattern there)

This is mainly for future reference, as I didn't tried to replicate this issue on a fresh environment or anything, but i think it may be useful for someone with similar issues.

The workaround is easy: disable http2 in nginx config.

@vlastikcz vlastikcz changed the title Upload issues with http2 Upload failing with http2 May 18, 2020
@fdintino
Copy link
Owner

fdintino commented Jun 1, 2020

Could you follow the instructions for enabling core dumps and then upload the core dump somewhere I can access (e.g. dropbox)? Or, at least, paste the backtrace from the core dump file (instructions here)

We currently use this module with http2 in production, and there are currently passing unit tests for http2 support, so there is some other variable involved that's triggering your issue.

@masoud-msk
Copy link

I had the same issue in production with both SSL and HTTP2 enabled. I disabled HTTP2 and the problem is gone for a few days.
I think you should either fix or mention this incompatibility in the docs for future reference.

@fdintino
Copy link
Owner

@masoud-msk As I stated in the previous comment, the issue isn't solely with http2. We use this module with http2 and we don't experience this segmentation fault. So there is some other variable that causes this bug, and until I know what that is I can't fix it. If you could take some time to share a core dump (instructions above in #132 (comment) ) I'd be able to identify and fix whatever the problem is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants