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

premature client disconnects #20

Open
bloodearnest opened this issue Sep 30, 2021 · 2 comments
Open

premature client disconnects #20

bloodearnest opened this issue Sep 30, 2021 · 2 comments

Comments

@bloodearnest
Copy link
Member

We're seeing frequent client disconnects.

The dokku2 nginx logs show premature close, e.g.

2021/09/28 13:19:27 [error] 29908#29908: *1727134 upstream prematurely closed connection while reading upstream, client: 62.253.26.158

It looks like we are seeing this with larger uploads, but not always.

I noticed that uploads were being buffered, so I turned that off in proxy nginx config, but it doesn't seem to have worked.

It may be something todo with our double nginx thing. It might be worth redesigning the proxy as just nginx config that's dropped in to dokku's nginx, rather than running a separate nginx as a dokku app

It seems elevated starting this week (27/09/2021), and there were some network changes on the 26th, so have email TPP

@sebbacon
Copy link

Is it worth investing in a script to attempt to reproduce the problem? We could provide it to TPP to run, as well, then.

@bloodearnest
Copy link
Member Author

So, it turns out I didn't turn off buffering properly, so I did that: https://github.com/opensafely-core/proxy/pull/19/files

I tested a manual push via the proxy, at file sizes up to 50mb (GH limit), and encounter no issues.

I successfully released the previously problematic 7.3mb html file, so this may be fixed, we'll see if users report any more issues.

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