search

numerique-gouv / helm-charts

Helm charts utilisé au sein de la DINUM
2 stars 2 forks source link

Grist: Raise proxy-body-size #16

Closed fflorent closed 7 months ago

fflorent commented 7 months ago

Contexte

Fixes #15

Lors d'un import de fichier sur Grist de plus d'1Mo, on a une erreur 413.

Solution proposée

On ajoute l'annotation suivante pour permettre d'augmenter le client_max_body_size côté Nginx : https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/#custom-max-body-size

rouja commented 7 months ago

Salut,

Je me demande si on ne devrait pas mettre cette annotation dans les values par defaut du chart helm plutôt ?

fflorent commented 7 months ago

On peut, je n'ai pas beaucoup d'avis à ce sujet. Après 500M, ça me semble raisonnable au vu des fichiers qui peuvent être créé avec Grist.

Je modifie ma PR.

fflorent commented 7 months ago

@rouja C'est poussé!