PR pour gestion des exceptions javascript levées par JSON.parse() sur un flux non conforme JSON (erreur HTTP diverses) #4848

Closed
g0uZ wants to merge 4 commits from g0uZ/bigup:master into master
g0uZ commented 1 year ago
There is no content yet.
g0uZ added 2 commits 1 year ago
b_b commented 1 year ago
Owner

Ça me semble bien, par contre il faudrait amha fusionner les deux commits et surtout leur ajouter un log plus sympa que "Mise à jour de 'javascript/bigup.js'" ^^

Ça me semble bien, par contre il faudrait amha fusionner les deux commits et surtout leur ajouter un log plus sympa que "Mise à jour de 'javascript/bigup.js'" ^^
b_b approved these changes 1 year ago
Owner

Je n’ai pas testé pour le moment, mais la proposition me semble correcte.
Je me demandais pour le nombre de tentative de retry, si 25 n’est pas un poil élevé, mais je n’ai aucune idée de la bonne pratique à ce sujet.

Je n’ai pas testé pour le moment, mais la proposition me semble correcte. Je me demandais pour le nombre de tentative de retry, si 25 n’est pas un poil élevé, mais je n’ai aucune idée de la bonne pratique à ce sujet.
marcimat approved these changes 12 months ago
g0uZ added 2 commits 12 months ago
Owner

Voir #4849 et #4850
Je ferme cette PR.

Voir #4849 et #4850 Je ferme cette PR.
marcimat closed this pull request 12 months ago

Reviewers

b_b approved these changes 1 year ago
marcimat approved these changes 12 months ago
Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.