gristlabs/grist-core

make limit on API request body size configurable

Open

#916 ouverte le 27 mars 2024

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)TypeScript (3 613 stars) (168 forks)batch import
good first issue

Description

There is a limit set on the maximum API request body size. That limit is currently hardcoded to 1mb (https://github.com/gristlabs/grist-help/pull/320#issuecomment-2021537352). It would be good to make this configurable, in the same way that related parameters like GRIST_MAX_UPLOAD_IMPORT_MB and GRIST_MAX_UPLOAD_ATTACHMENT_MB are.

Guide contributeur

Stack technique
typescriptnodejs
Domaine
backend
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
familiarity with TypeScriptbasic knowledge of Grist's configuration
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Look at the hardcoded limit referenced in the issue (https://github.com/gristlabs/grist help/pull/320#issuecomment 2021537352). Examine how similar parameters like GRIST MAX UPLOAD IMPORT MB and GRIST MAX UPLOAD ATTACHMENT MB are implemented and made configurable. The goal is to introduce a new environment variable (e.g., GRIST MAX REQUEST BODY MB) and replace the hardcoded value with a check that reads this variable, falling back to 1MB if not set.