Bump matrix client_max_body_size to 1GB #140

Merged
samsapti merged 2 commits from matrix-client-max-body-size into main 2023-01-06 13:22:58 +00:00
Owner

I experience videos don't always work, and I suspect this could be the reason.

I experience videos don't always work, and I suspect this could be the reason.
reynir added 1 commit 2022-12-17 20:29:10 +00:00
reynir added 1 commit 2022-12-17 20:43:28 +00:00
samsapti requested changes 2022-12-17 21:01:06 +00:00
@ -1,2 +1,2 @@
listen 8008;
client_max_body_size 50M; # default is 1M
client_max_body_size 1G; # default is 1M
Owner

We should revert this.

We should revert this.
Author
Owner

Why?

Why?
Owner

Because I think 50M is sufficient. Additionally, it wasn't that change that fixed the issue. Let's leave it at 50M if it doesn't create any issues.

Because I think 50M is sufficient. Additionally, it wasn't that change that fixed the issue. Let's leave it at 50M if it doesn't create any issues.
Owner

Nevermind actually, I think it's fine.

Nevermind actually, I think it's fine.
samsapti marked this conversation as resolved
@ -1 +1 @@
client_max_body_size 50M; # default is 1M
client_max_body_size 1G; # default is 1M
Owner

This too.

This too.
samsapti marked this conversation as resolved
samsapti added the
Existing Service
label 2022-12-20 22:08:04 +00:00
samsapti approved these changes 2023-01-06 13:19:53 +00:00
samsapti merged commit 035c683f67 into main 2023-01-06 13:22:58 +00:00
samsapti deleted branch matrix-client-max-body-size 2023-01-06 13:22:59 +00:00
Sign in to join this conversation.
No description provided.