Skip to content

Commit 2defef3

Browse files
committed
Refactor SMTP name to always have email- prefix for consistency.
1 parent 24af0bc commit 2defef3

36 files changed

+40
-35
lines changed

cmd/init.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -573,7 +573,7 @@ func initSMTPMessengers() []manager.Messenger {
573573
// If the server has a name, initialize it as a standalone e-mail messenger
574574
// allowing campaigns to select individual SMTPs. In the UI and config, it'll appear as `email / $name`.
575575
if s.Name != "" {
576-
msgr, err := email.New(fmt.Sprintf("%s / %s", email.MessengerName, s.Name), s)
576+
msgr, err := email.New(s.Name, s)
577577
if err != nil {
578578
lo.Fatalf("error initializing e-mail messenger: %v", err)
579579
}

cmd/settings.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,10 @@ func handleUpdateSettings(c echo.Context) error {
111111
// Sanitize and normalize the SMTP server name.
112112
name := reAlphaNum.ReplaceAllString(strings.ToLower(strings.TrimSpace(s.Name)), "-")
113113
if name != "" {
114+
if !strings.HasPrefix(name, "email-") {
115+
name = "email-" + name
116+
}
117+
114118
if _, ok := names[name]; ok {
115119
return echo.NewHTTPError(http.StatusBadRequest,
116120
app.i18n.Ts("settings.duplicateMessengerName", "name", name))

frontend/src/views/Campaign.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -651,11 +651,11 @@ export default Vue.extend({
651651
},
652652
653653
emailMessengers() {
654-
return ['email', ...this.serverConfig.messengers.filter((m) => m.startsWith('email /'))];
654+
return ['email', ...this.serverConfig.messengers.filter((m) => m.startsWith('email-'))];
655655
},
656656
657657
otherMessengers() {
658-
return this.serverConfig.messengers.filter((m) => m !== 'email' && !m.startsWith('email /'));
658+
return this.serverConfig.messengers.filter((m) => m !== 'email' && !m.startsWith('email-'));
659659
},
660660
},
661661

frontend/src/views/settings/smtp.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@
145145
<div class="column is-6">
146146
<b-field :label="$t('globals.fields.name')" label-position="on-border"
147147
:message="$t('settings.mailserver.nameHelp')">
148-
<b-input v-model="item.name" name="name" placeholder="" :maxlength="100" />
148+
<b-input v-model="item.name" name="name" placeholder="email-primary" :maxlength="100" />
149149
</b-field>
150150
</div>
151151
</div>

i18n/ca.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -430,7 +430,7 @@
430430
"settings.mailserver.idleTimeoutHelp": "Temps d'inactivitat per esperar una nova activitat en una connexió abans de tancar-la i eliminar-la de la grup (s per segon, m per minut).",
431431
"settings.mailserver.maxConns": "Connexions màximes",
432432
"settings.mailserver.maxConnsHelp": "Màxim de connexions concurrents al servidor.",
433-
"settings.mailserver.nameHelp": "Nom opcional únic per al servidor SMTP. Establir això permet seleccionar específicament el servidor per a una campanya. Exemple: primary-server. Alfanumèric / guionet.",
433+
"settings.mailserver.nameHelp": "Nom opcional únic per al servidor SMTP. Ha de tenir el prefix email-. Establir això permet seleccionar específicament el servidor per a una campanya. Ex: email-primary-server. Alfanumèric / guionet.",
434434
"settings.mailserver.password": "Contrasenya",
435435
"settings.mailserver.passwordHelp": "Fes intro per canviar",
436436
"settings.mailserver.port": "Port del servidor de correu",

i18n/cs-cz.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -435,7 +435,7 @@
435435
"settings.mailserver.maxConns": "Maximální počet připojení",
436436
"settings.mailserver.maxConnsHelp": "Maximální počet souběžných připojení k serveru.",
437437
"settings.mailserver.name": "Jméno",
438-
"settings.mailserver.nameHelp": "Volitelný jedinečný název pro SMTP server. Nastavením tohoto parametru je možné vybrat daný server pro danou kampaň. Např.: primary-server. Alfanumerické / pomlčka.",
438+
"settings.mailserver.nameHelp": "Volitelný jedinečný název pro SMTP server. Musí mít předponu email-. Nastavení tohoto umožňuje vybrat server specificky pro kampaň, např. email-primary-server. Alfanumerické / pomlčka.",
439439
"settings.mailserver.password": "Heslo",
440440
"settings.mailserver.passwordHelp": "Klávesou Enter zadejte změnu",
441441
"settings.mailserver.port": "Port",

i18n/cy.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -435,7 +435,7 @@
435435
"settings.mailserver.maxConns": "Uchafswm nifer y cysylltiadau",
436436
"settings.mailserver.maxConnsHelp": "Uchafswm nifer y cysylltiadau cydamserol â'r gweinydd",
437437
"settings.mailserver.name": "Enw",
438-
"settings.mailserver.nameHelp": "Enw unigryw dewisol ar gyfer y gweinydd SMTP. Trwy osod hwn, mae'n caniatáu i'r gweinydd gael ei ddewis yn benodol ar gyfer ymgyrch. e.e.: gwefr-prif. Alffaniwmerig / llinell.",
438+
"settings.mailserver.nameHelp": "Enw unigryw dewisol ar gyfer y gweinydd SMTP. Rhaid i'r gweinydd gael y rhagddodiad email-. Mae gosod hyn yn caniatáu i'r gweinydd gael ei ddewis yn benodol ar gyfer ymgyrch. e.e: gweinydd-prif-ebost. Allweddolalfanumerig / llinell.",
439439
"settings.mailserver.password": "Cyfrinair",
440440
"settings.mailserver.passwordHelp": "Pwyswch enter i'w newid",
441441
"settings.mailserver.port": "Porth",

i18n/da.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -431,7 +431,7 @@
431431
"settings.mailserver.maxConns": "Maks. tilslutninger",
432432
"settings.mailserver.maxConnsHelp": "Maksimalt antal samtidige forbindelser til serveren.",
433433
"settings.mailserver.name": "Navn",
434-
"settings.mailserver.nameHelp": "Valgfrit unikt navn til SMTP-serveren. Ved at angive dette kan serveren vælges specifikt til en kampagne. f.eks.: primær-server. Alfanumerisk / bindestreg.",
434+
"settings.mailserver.nameHelp": "Valgfri unikt navn til SMTP-serveren. Skal have præfixet email-. På denne måde kan serveren vælges specifikt til en kampagne. Eksempelvis: email-primary-server. Alfanumerisk / bindestreg.",
435435
"settings.mailserver.password": "Kodeord",
436436
"settings.mailserver.passwordHelp": "Indtast for at ændre",
437437
"settings.mailserver.port": "Port",

i18n/de.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -431,7 +431,7 @@
431431
"settings.mailserver.maxConns": "Max. Verbindungen",
432432
"settings.mailserver.maxConnsHelp": "Maximale gleichzeitige Verbindungen zum SMTP Server",
433433
"settings.mailserver.name": "Name",
434-
"settings.mailserver.nameHelp": "Optionaler eindeutiger Name für den SMTP-Server. Durch das Festlegen dieses Namens kann der Server speziell für eine Kampagne ausgewählt werden, z.B. primärer-server. Alphanumerisch / Bindestrich.",
434+
"settings.mailserver.nameHelp": "Optionaler eindeutiger Name für den SMTP-Server. Muss den Präfix email- haben. Durch die Festlegung kann der Server speziell für eine Kampagne ausgewählt werden. Z. B .: email-primary-server. Alphanumerisch / Bindestrich.",
435435
"settings.mailserver.password": "Passwort",
436436
"settings.mailserver.passwordHelp": "Gib dein Passwort ein, um es zu ändern",
437437
"settings.mailserver.port": "Port",

i18n/el.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -431,7 +431,7 @@
431431
"settings.mailserver.maxConns": "Μέγιστες συνδέσεις",
432432
"settings.mailserver.maxConnsHelp": "Μέγιστες ταυτόχρονες συνδέσεις στο διακομιστή.",
433433
"settings.mailserver.name": "Ονομα",
434-
"settings.mailserver.nameHelp": "Προαιρετικό μοναδικό όνομα για τον διακομιστή SMTP. Η ρύθμιση αυτή επιτρέπει την επιλογή συγκεκριμένου διακομιστή για μία καμπάνια. π.χ.: primary-server. Αλφαριθμητικά / παύλα.",
434+
"settings.mailserver.nameHelp": "Προαιρετικό μοναδικό όνομα για τον διακομιστή SMTP. Πρέπει να έχει το πρόθεμα email-. Η οριστική ρύθμιση αυτής της τιμής επιτρέπει την επιλογή του διακομιστή συγκεκριμένα για μια καμπάνια. π.χ .: email-primary-server. Αλφαριθμητικό / παύλα.",
435435
"settings.mailserver.password": "Κωδικός πρόσβασης",
436436
"settings.mailserver.passwordHelp": "Enter για να το αλλάξετε",
437437
"settings.mailserver.port": "Θύρα",

0 commit comments

Comments
 (0)