data.coop/ansible
8
3
Fork 7
Code Issues 49 Pull requests 9 Projects 1 Releases Wiki Activity

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
Conversation 0 Commits 2 Files changed 3 +3 -3
3 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
roles/docker/files/configs/matrix/homeserver.yaml.j2
View file

@ -416,7 +416,7 @@ uploads_path: "/data/uploads"
# The largest allowed upload size in bytes
#
max_upload_size: "50M"
max_upload_size: "512M"
# Maximum number of pixels that will be thumbnailed
#

2
roles/docker/files/configs/matrix/vhost-matrix
View file

@ -1,2 +1,2 @@
listen 8008;
client_max_body_size 50M; # default is 1M
client_max_body_size 1G; # default is 1M
samsapti marked this conversation as resolved
samsapti commented 2022-12-17 21:00:46 +00:00
Review
Copy link

We should revert this.

We should revert this.
reynir commented 2022-12-17 21:14:22 +00:00
Review
Copy link

Why?

Why?
samsapti commented 2022-12-18 14:47:36 +00:00
Review
Copy link

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.
samsapti commented 2023-01-06 13:19:41 +00:00
Review
Copy link

Nevermind actually, I think it's fine.

Nevermind actually, I think it's fine.

2
roles/docker/files/configs/matrix/vhost-riot
View file

@ -1 +1 @@
client_max_body_size 50M; # default is 1M
client_max_body_size 1G; # default is 1M
samsapti marked this conversation as resolved
samsapti commented 2022-12-17 21:00:56 +00:00
Review
Copy link

This too.

This too.