413 (Content Too Large) when uploading a complex scene from within Open Brush

andybak commented 4 months ago

VrAssetServiceException: POST failed
HTTP error 413
URI: POST https://icosa-api.ixxy.co.uk/assets UploadHandlerRaw archive.zip
Headers: Date: Wed, 22 May 2024 13:29:27 GMT
Content-Type: text/html
Transfer-Encoding: chunked
Connection: keep-alive
CF-Cache-Status: DYNAMIC
Report-To: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v4?s=8rFDUgyT2%2BjkHOyShMmlZ6UCTG0nuOax%2BTm5nZQs1O1tzk%2FNJ%2FnM0K6w52UiieFfV1aFVparmGE0mPddXZQOdBfptf%2BdQrXoBtZFUepLrJ6sOxqZBUfCYLSHts0aabYtg440TxOmjHammFl7VO6HZ3F%2B4g%3D%3D"}],"group":"cf-nel","max_age":604800}
NEL: {"success_fraction":0,"report_to":"cf-nel","max_age":604800}
Server: cloudflare
CF-RAY: 887d27b2ee9e948d-LHR
alt-svc: h3=":443"; ma=86400

Body: <html>
<head><title>413 Request Entity Too Large</title></head>
<body>
<center><h1>413 Request Entity Too Large</h1></center>
<hr><center>nginx/1.25.5</center>
</body>
</html>

jamesrampton commented 4 months ago

@andybak What's a realistic scene size? Looks like this is nginx crying so I'll up the client_max_body_size there first.

Even better, an example upload that can reproduce this would be much appreciated.

andybak commented 4 months ago

This failed on a 10mb zip but sketches can get waaaay bigger than that.

I'm presuming this will tie up an nginx worker for the duration of the upload? Which I presume is bad?

mikeskydev commented 4 months ago

this was my strategy with fastapi spawning a number of workers, noticed it fairly quickly when we went to production the first time!

jamesrampton commented 4 months ago

Just for the time being, so we can test, I've upped the max size to 100M.

jamesrampton commented 4 months ago

@andybak it's now 500M

icosa-foundation / icosa-gallery

413 (Content Too Large) when uploading a complex scene from within Open Brush #4