smee.io icon indicating copy to clipboard operation
smee.io copied to clipboard

big requests lead to 413 PayloadTooLargeError

Open gerrnot opened this issue 2 years ago • 1 comments

Hi,

A colleague pushed a change with many changed files/folders (87) to GitHub.

As the list of files is part of the webhook event, this had the effect that a quite large webhook event (request body) was delivered to smee.io.

The webhook type was a regular json and had a size of 156484 bytes.

I think it would be good for stability to also support larger pushes and increase the size to maybe 1MB.

Workaround: Push a smaller commit afterwards, then only the changes of the newer commit are part of the event.

Regards, Gernot

gerrnot avatar Jul 22 '22 14:07 gerrnot

GitHub can send payloads of up to 25MB 1

Smee is not meant to be used as a production service, it's to help facilitate during development.

Now, 25MB is definitely out of the question, 1MB payloads sound reasonable

wolfy1339 avatar Jul 23 '24 22:07 wolfy1339