entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 37 Releases Activity

workflows: reject mails over 50MB (#50002) #563

Merged
fpeters merged 1 commits from wip/50002-reject-big-mails into main 2023-08-11 00:31:34 +02:00
Conversation 0 Commits 1 Files Changed 4 +62 -19
fpeters commented 2023-08-06 12:17:47 +02:00
Owner
Copy Link
No description provided.
fpeters changed title from WIP: workflows: reject mails over 50MB (#50002) to workflows: reject mails over 50MB (#50002) 2023-08-07 12:24:50 +02:00
lguerin approved these changes 2023-08-07 13:21:13 +02:00
tnoel approved these changes 2023-08-07 14:09:55 +02:00
tnoel left a comment
Owner
Copy Link

J'accepte aussi comme Zébuline, mais je me permets une petite remarque si tu as le temps, pour clarifier l'erreur quand elle arrivera (et nous éviter une question au support)

J'accepte aussi comme Zébuline, mais je me permets une petite remarque si tu as le temps, pour clarifier l'erreur quand elle arrivera (et nous éviter une question au support)
wcs/qommon/errors.py Outdated
@ -76,6 +76,10 @@ class EmailError(Exception):
pass
class TooBigEmailError(EmailError):
tnoel commented 2023-08-07 14:09:09 +02:00
Owner
Copy Link

Peut-être qu'on pourrait ici enregistrer la longueur du mail refusé (et la longueur max acceptée) afin que ça puisse être affiché dans l'erreur enregistrée, pour mieux comprendre la cause.

Peut-être qu'on pourrait ici enregistrer la longueur du mail refusé (et la longueur max acceptée) afin que ça puisse être affiché dans l'erreur enregistrée, pour mieux comprendre la cause.
fpeters commented 2023-08-07 16:02:34 +02:00
Author
Owner
Copy Link

J'avais une idée comme ça au début mais le modlèle LoggedError ne permet pas d'ajouter des informations supplémentaires et je ne voulais pas me lancer dans des développements là-dedans. (et garder le même message de base pour que ça soit mergé en une seule erreur). Je verrai pour faire évoluer ça plus tard, selon les retours.

J'avais une idée comme ça au début mais le modlèle LoggedError ne permet pas d'ajouter des informations supplémentaires et je ne voulais pas me lancer dans des développements là-dedans. (et garder le même message de base pour que ça soit mergé en une seule erreur). Je verrai pour faire évoluer ça plus tard, selon les retours.
fpeters force-pushed wip/50002-reject-big-mails from 2c83d262ee to 14a763c63a 2023-08-07 16:14:37 +02:00 Compare
fpeters merged commit 9c9b2738c4 into main 2023-08-11 00:31:34 +02:00
fpeters deleted branch wip/50002-reject-big-mails 2023-08-11 00:31:35 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#563
No description provided.
Delete Branch "wip/50002-reject-big-mails"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?