Cadoles/fake-smtp
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 added 1 commit 2025-03-13 11:21:42 +01:00
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
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?