Cadoles/fake-smtp
22
0
Fork 0
Code Issues 2 Pull Requests 1 Releases Wiki Activity

add email rotation regarding a limit #9

Merged
wpetit merged 1 commits from rotate-email into develop 2025-03-13 13:37:36 +01:00
Conversation 3 Commits 1 Files Changed 5 +66 -2
mlamalle commented 2025-03-13 11:21:41 +01:00
Owner
Copy Link

Changelog:

  • Ajout de la variable d'environnement FAKESMTP_DATA_MAX_EMAIL pour limiter le nombre maximum d'emails stockés.
  • Mise à jour de la configuration pour inclure MaxEmail dans DataConfig.
  • Ajout de la gestion de la commande RotateEmailRequest pour gérer la rotation des emails en fonction de MaxEmail.
  • Modification du fichier smtp.go pour exécuter la commande de rotation des emails si MaxEmail est défini. -
Changelog: - Ajout de la variable d'environnement `FAKESMTP_DATA_MAX_EMAIL` pour limiter le nombre maximum d'emails stockés. - Mise à jour de la configuration pour inclure `MaxEmail` dans `DataConfig`. - Ajout de la gestion de la commande `RotateEmailRequest` pour gérer la rotation des emails en fonction de `MaxEmail`. - Modification du fichier `smtp.go` pour exécuter la commande de rotation des emails si `MaxEmail` est défini. -
mlamalle force-pushed rotate-email from a990551ec8 to 730e9f8bde 2025-03-13 12:00:23 +01:00 Compare
wpetit requested changes 2025-03-13 12:41:15 +01:00
Dismissed
internal/command/rotate_email.go Outdated
@ -0,0 +30,4 @@
return errors.Wrap(err, "could not retrieve storm service")
}
emailcount, err := db.Select().Count(&model.Email{})
wpetit commented 2025-03-13 12:40:17 +01:00
Owner
Copy Link

Utiliser le camelcase ? emailCount ?

Utiliser le camelcase ? `emailCount` ?
mlamalle marked this conversation as resolved
internal/command/rotate_email.go Outdated
@ -0,0 +36,4 @@
}
if emailcount >= req.MaxEmail {
var intdiff = emailcount - (req.MaxEmail - 1)
wpetit commented 2025-03-13 12:39:46 +01:00
Owner
Copy Link

Juste renommer la variable en diff, pas besoin d'intégrer le type, on est pas en JS :P

Juste renommer la variable en `diff`, pas besoin d'intégrer le type, on est pas en JS :P
mlamalle marked this conversation as resolved
mlamalle force-pushed rotate-email from 730e9f8bde to 0c23304d34 2025-03-13 13:33:59 +01:00 Compare
lgourvenec commented 2025-03-13 13:36:41 +01:00
Owner
Copy Link

closes #7

closes https://forge.cadoles.com/Cadoles/fake-smtp/issues/7
wpetit approved these changes 2025-03-13 13:37:31 +01:00
wpetit merged commit 6a46581c4f into develop 2025-03-13 13:37:36 +01:00
wpetit deleted branch rotate-email 2025-03-13 13:37:38 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
wpetit
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
aardouin AdminInfra afornerot bbohard bgaude cmsassot crappeneau gdupont idargaud jenkins kipp lfornerot lgourvenec lseys mlamalle pcaseiro rmasson tcornaut vcarroy vfebvre wpetit yszemro
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Cadoles/fake-smtp#9
No description provided.
Delete Branch "rotate-email"

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?