sifive / wake

The SiFive wake build tool
Other
86 stars 28 forks source link

rsc: Log failures to upload blob #1623

Closed V-FEXrt closed 3 months ago

V-FEXrt commented 3 months ago

We've observed blob uploads that silently failed which caused a delay is response time. This updates the POST /blob route to log an error when blob upload fails for some reason

V-FEXrt commented 3 months ago

Ah that's intentional. That only happens when the client make a bad request, its nothing related to the server.

My thought there was that we shouldn't allow a misbehaving client to spam GBs of data into the server logs

colinschmidt commented 3 months ago

Ah that's intentional. That only happens when the client make a bad request, its nothing related to the server.

My thought there was that we shouldn't allow a misbehaving client to spam GBs of data into the server logs

So that error would show up in the client log? If so thats ok.

V-FEXrt commented 3 months ago

So that error would show up in the client log? If so thats ok.

Yep. It's pretty much a "client developer only" error because its the kind of thing that once you get it right the code never changes and you never hit that failure case again. Its the same kind of error as checking the shape of the POST'd JSON. You might have the client shape wrong a few times during development but once you get the shape right its always correct until the server changes it