Skip to content

Commit

Permalink
Refactor SMTP name to always have email- prefix for consistency.
Browse files Browse the repository at this point in the history
  • Loading branch information
knadh committed Feb 14, 2025
1 parent 24af0bc commit 2defef3
Show file tree
Hide file tree
Showing 36 changed files with 40 additions and 35 deletions.
2 changes: 1 addition & 1 deletion cmd/init.go
Original file line number Diff line number Diff line change
Expand Up @@ -573,7 +573,7 @@ func initSMTPMessengers() []manager.Messenger {
// If the server has a name, initialize it as a standalone e-mail messenger
// allowing campaigns to select individual SMTPs. In the UI and config, it'll appear as `email / $name`.
if s.Name != "" {
msgr, err := email.New(fmt.Sprintf("%s / %s", email.MessengerName, s.Name), s)
msgr, err := email.New(s.Name, s)
if err != nil {
lo.Fatalf("error initializing e-mail messenger: %v", err)
}
Expand Down
4 changes: 4 additions & 0 deletions cmd/settings.go
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,10 @@ func handleUpdateSettings(c echo.Context) error {
// Sanitize and normalize the SMTP server name.
name := reAlphaNum.ReplaceAllString(strings.ToLower(strings.TrimSpace(s.Name)), "-")
if name != "" {
if !strings.HasPrefix(name, "email-") {
name = "email-" + name
}

if _, ok := names[name]; ok {
return echo.NewHTTPError(http.StatusBadRequest,
app.i18n.Ts("settings.duplicateMessengerName", "name", name))
Expand Down
4 changes: 2 additions & 2 deletions frontend/src/views/Campaign.vue
Original file line number Diff line number Diff line change
Expand Up @@ -651,11 +651,11 @@ export default Vue.extend({
},
emailMessengers() {
return ['email', ...this.serverConfig.messengers.filter((m) => m.startsWith('email /'))];
return ['email', ...this.serverConfig.messengers.filter((m) => m.startsWith('email-'))];
},
otherMessengers() {
return this.serverConfig.messengers.filter((m) => m !== 'email' && !m.startsWith('email /'));
return this.serverConfig.messengers.filter((m) => m !== 'email' && !m.startsWith('email-'));
},
},
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/views/settings/smtp.vue
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@
<div class="column is-6">
<b-field :label="$t('globals.fields.name')" label-position="on-border"
:message="$t('settings.mailserver.nameHelp')">
<b-input v-model="item.name" name="name" placeholder="" :maxlength="100" />
<b-input v-model="item.name" name="name" placeholder="email-primary" :maxlength="100" />
</b-field>
</div>
</div>
Expand Down
2 changes: 1 addition & 1 deletion i18n/ca.json
Original file line number Diff line number Diff line change
Expand Up @@ -430,7 +430,7 @@
"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).",
"settings.mailserver.maxConns": "Connexions màximes",
"settings.mailserver.maxConnsHelp": "Màxim de connexions concurrents al servidor.",
"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.",
"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.",
"settings.mailserver.password": "Contrasenya",
"settings.mailserver.passwordHelp": "Fes intro per canviar",
"settings.mailserver.port": "Port del servidor de correu",
Expand Down
2 changes: 1 addition & 1 deletion i18n/cs-cz.json
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@
"settings.mailserver.maxConns": "Maximální počet připojení",
"settings.mailserver.maxConnsHelp": "Maximální počet souběžných připojení k serveru.",
"settings.mailserver.name": "Jméno",
"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.",
"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.",
"settings.mailserver.password": "Heslo",
"settings.mailserver.passwordHelp": "Klávesou Enter zadejte změnu",
"settings.mailserver.port": "Port",
Expand Down
2 changes: 1 addition & 1 deletion i18n/cy.json
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@
"settings.mailserver.maxConns": "Uchafswm nifer y cysylltiadau",
"settings.mailserver.maxConnsHelp": "Uchafswm nifer y cysylltiadau cydamserol â'r gweinydd",
"settings.mailserver.name": "Enw",
"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.",
"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.",
"settings.mailserver.password": "Cyfrinair",
"settings.mailserver.passwordHelp": "Pwyswch enter i'w newid",
"settings.mailserver.port": "Porth",
Expand Down
2 changes: 1 addition & 1 deletion i18n/da.json
Original file line number Diff line number Diff line change
Expand Up @@ -431,7 +431,7 @@
"settings.mailserver.maxConns": "Maks. tilslutninger",
"settings.mailserver.maxConnsHelp": "Maksimalt antal samtidige forbindelser til serveren.",
"settings.mailserver.name": "Navn",
"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.",
"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.",
"settings.mailserver.password": "Kodeord",
"settings.mailserver.passwordHelp": "Indtast for at ændre",
"settings.mailserver.port": "Port",
Expand Down
2 changes: 1 addition & 1 deletion i18n/de.json
Original file line number Diff line number Diff line change
Expand Up @@ -431,7 +431,7 @@
"settings.mailserver.maxConns": "Max. Verbindungen",
"settings.mailserver.maxConnsHelp": "Maximale gleichzeitige Verbindungen zum SMTP Server",
"settings.mailserver.name": "Name",
"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.",
"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.",
"settings.mailserver.password": "Passwort",
"settings.mailserver.passwordHelp": "Gib dein Passwort ein, um es zu ändern",
"settings.mailserver.port": "Port",
Expand Down
2 changes: 1 addition & 1 deletion i18n/el.json
Original file line number Diff line number Diff line change
Expand Up @@ -431,7 +431,7 @@
"settings.mailserver.maxConns": "Μέγιστες συνδέσεις",
"settings.mailserver.maxConnsHelp": "Μέγιστες ταυτόχρονες συνδέσεις στο διακομιστή.",
"settings.mailserver.name": "Ονομα",
"settings.mailserver.nameHelp": "Προαιρετικό μοναδικό όνομα για τον διακομιστή SMTP. Η ρύθμιση αυτή επιτρέπει την επιλογή συγκεκριμένου διακομιστή για μία καμπάνια. π.χ.: primary-server. Αλφαριθμητικά / παύλα.",
"settings.mailserver.nameHelp": "Προαιρετικό μοναδικό όνομα για τον διακομιστή SMTP. Πρέπει να έχει το πρόθεμα email-. Η οριστική ρύθμιση αυτής της τιμής επιτρέπει την επιλογή του διακομιστή συγκεκριμένα για μια καμπάνια. π.χ .: email-primary-server. Αλφαριθμητικό / παύλα.",
"settings.mailserver.password": "Κωδικός πρόσβασης",
"settings.mailserver.passwordHelp": "Enter για να το αλλάξετε",
"settings.mailserver.port": "Θύρα",
Expand Down
2 changes: 1 addition & 1 deletion i18n/en.json
Original file line number Diff line number Diff line change
Expand Up @@ -430,7 +430,7 @@
"settings.mailserver.idleTimeoutHelp": "Time to wait for new activity on a connection before closing it and removing it from the pool (s for second, m for minute).",
"settings.mailserver.maxConns": "Max. connections",
"settings.mailserver.maxConnsHelp": "Maximum concurrent connections to the server.",
"settings.mailserver.nameHelp": "Optional unique name for the SMTP server. Setting this allows the server to be specifically selected for a campaign. eg: primary-server. Alphanumeric / dash.",
"settings.mailserver.nameHelp": "Optional unique name for the SMTP server. Must have the prefix email-. Setting this allows the server to be specifically selected for a campaign. eg: email-primary-server. Alphanumeric / dash.",
"settings.mailserver.password": "Password",
"settings.mailserver.passwordHelp": "Enter to change",
"settings.mailserver.port": "Port",
Expand Down
2 changes: 1 addition & 1 deletion i18n/eo.json
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@
"settings.mailserver.maxConns": "Connexions màximes",
"settings.mailserver.maxConnsHelp": "Màxim de connexions concurrents al servidor.",
"settings.mailserver.name": "Nomo",
"settings.mailserver.nameHelp": "Opcia unika nomo por la SMTP-servilo. La difino permesas elekti la servilon specife por kampanjo. Ekz.: primara-servilo. Alfa-numera / streko.",
"settings.mailserver.nameHelp": "Opcia unika nomo por la SMTP-servilo. Devas havi la antaŭaldonaĵon 'email-'. Tio ebligas elekti la servilon specife por kampanjo. Ekz.: email-ĉef-servilo. Alfanumerika / streketo.",
"settings.mailserver.password": "Contrasenya",
"settings.mailserver.passwordHelp": "Fes intro per canviar",
"settings.mailserver.port": "Pordo",
Expand Down
2 changes: 1 addition & 1 deletion i18n/es.json
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,7 @@
"settings.mailserver.maxConns": "Conexiones máximas",
"settings.mailserver.maxConnsHelp": "Número máximo de conexiones concurrentes hacia el servidor SMTP.",
"settings.mailserver.name": "Nombre",
"settings.mailserver.nameHelp": "Nombre opcional único para el servidor SMTP. Configurar esto permite seleccionar específicamente el servidor para una campaña. Ejemplo: servidor-principal. Alfanumérico / guión.",
"settings.mailserver.nameHelp": "Nombre único opcional para el servidor SMTP. Debe tener el prefijo email-. Configurar esto permite seleccionar específicamente el servidor para una campaña, por ejemplo: email-primary-server. Alfanumérico / guión.",
"settings.mailserver.password": "Contraseña",
"settings.mailserver.passwordHelp": "Ingresar contraseña para cambiar",
"settings.mailserver.port": "Puerto",
Expand Down
2 changes: 1 addition & 1 deletion i18n/fi.json
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@
"settings.mailserver.maxConns": "Maksimiyhteydet",
"settings.mailserver.maxConnsHelp": "Kerralla samaan aikaan avoimet yhteydet palvelimeen.",
"settings.mailserver.name": "Nimi",
"settings.mailserver.nameHelp": "Vaihtoehtoinen uniikki nimi SMTP-palvelimelle. Tämän asettaminen mahdollistaa palvelimen erityisen valinnan kampanjalle. Esim: primary-server. Aakkosnumeeriset / viivalla erotetut merkit.",
"settings.mailserver.nameHelp": "Valinnainen ainutlaatuinen nimi SMTP-palvelimelle. Nimen tulee alkaa email- etuliitteellä. Tämän asetuksen avulla palvelin voidaan valita erityisesti kampanjaan. Esim. email-primary-server. Aakkosnumeeriset merkit / väliviiva sallittuja.",
"settings.mailserver.password": "Salasana",
"settings.mailserver.passwordHelp": "Syötä muuttaaksesi",
"settings.mailserver.port": "Portti",
Expand Down
2 changes: 1 addition & 1 deletion i18n/fr-CA.json
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,7 @@
"settings.mailserver.maxConns": "Nb. de connexions max.",
"settings.mailserver.maxConnsHelp": "Nombre maximum de connexions simultanées au serveur SMTP",
"settings.mailserver.name": "Nom",
"settings.mailserver.nameHelp": "Nom facultatif unique pour le serveur SMTP. Le paramétrer permet de le sélectionner spécifiquement pour une campagne. Par exemple : primary-server. Alphanumérique / tiret.",
"settings.mailserver.nameHelp": "Nom unique facultatif pour le serveur SMTP. Doit avoir le préfixe email-. Le fait de définir cela permet de sélectionner spécifiquement le serveur pour une campagne, par exemple : email-primary-server. Alphanumérique / tiret.",
"settings.mailserver.password": "Mot de passe",
"settings.mailserver.passwordHelp": "Entrez un nouveau mot de passe si vous souhaitez le modifier",
"settings.mailserver.port": "Port",
Expand Down
2 changes: 1 addition & 1 deletion i18n/fr.json
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,7 @@
"settings.mailserver.maxConns": "Nb. de connexions max.",
"settings.mailserver.maxConnsHelp": "Nombre maximum de connexions simultanées au serveur SMTP",
"settings.mailserver.name": "Nom",
"settings.mailserver.nameHelp": "Nom facultatif unique pour le serveur SMTP. Le définir permet de sélectionner spécifiquement ce serveur pour une campagne. Par exemple : primary-server. Alphanumérique / tiret.",
"settings.mailserver.nameHelp": "Nom facultatif unique pour le serveur SMTP. Doit avoir le préfixe email-. En le définissant, le serveur peut être spécifiquement sélectionné pour une campagne, par exemple email-primary-server. Alphanumérique / tiret.",
"settings.mailserver.password": "Mot de passe",
"settings.mailserver.passwordHelp": "Entrez un nouveau mot de passe si vous souhaitez le modifier",
"settings.mailserver.port": "Port",
Expand Down
2 changes: 1 addition & 1 deletion i18n/he.json
Original file line number Diff line number Diff line change
Expand Up @@ -431,7 +431,7 @@
"settings.mailserver.maxConns": "מקסימום חיבורים",
"settings.mailserver.maxConnsHelp": "למספר החיבורים המרובים הממוקדים לשרת.",
"settings.mailserver.name": "שֵׁם",
"settings.mailserver.nameHelp": "שם אופציונלי ייחודי עבור שרת SMTP. הגדרת זאת מאפשרת בחירה ממוקדת של השרת לקמפיין מסוים. לדוג' primary-server. מותרים אותיות ומקף.",
"settings.mailserver.nameHelp": "שם ייחודי אופציונלי עבור שרת SMTP. חייב להכיל את הקידומת email-. הגדרה זו מאפשרת לבחור את השרת למסע דואר מסוים. לדוגמה: email-primary-server. אותיות ומקף בלבד.",
"settings.mailserver.password": "סיסמא",
"settings.mailserver.passwordHelp": "יש להזין כדי לשנות",
"settings.mailserver.port": "פורט",
Expand Down
2 changes: 1 addition & 1 deletion i18n/hu.json
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@
"settings.mailserver.maxConns": "Kapcsolatok",
"settings.mailserver.maxConnsHelp": "Egyidejű kapcsolatok maximális száma.",
"settings.mailserver.name": "Név",
"settings.mailserver.nameHelp": "Választható egyedi név az SMTP szerverhez. A beállítás lehetővé teszi, hogy a szervert kifejezetten egy kampányhoz válasszák ki. pl.: primary-server. Alfanumerikus karakterek / kötőjelek engedélyezettek.",
"settings.mailserver.nameHelp": "Opcionális egyedi név az SMTP kiszolgálóhoz. Az email- előtaggal kell rendelkeznie. A beállítás lehetővé teszi, hogy a kiszolgálót kifejezetten kiválasszák egy kampányhoz. Példa: email-fő-szerver. Alfanumerikus / kötőjeles karakterek engedélyezettek.",
"settings.mailserver.password": "Jelszó",
"settings.mailserver.passwordHelp": "írja be a jelszót",
"settings.mailserver.port": "Port",
Expand Down
2 changes: 1 addition & 1 deletion i18n/it.json
Original file line number Diff line number Diff line change
Expand Up @@ -431,7 +431,7 @@
"settings.mailserver.maxConns": "Nb. connessioni max.",
"settings.mailserver.maxConnsHelp": "Numero massimo di connessioni simultanee al server SMTP.",
"settings.mailserver.name": "Nome",
"settings.mailserver.nameHelp": "Nome opzionale univoco per il server SMTP. Impostando questo campo, sarà possibile selezionare specificamente il server per una campagna. Esempio: primary-server. Alfanumerico / trattino.",
"settings.mailserver.nameHelp": "Nome opzionale univoco per il server SMTP. Deve avere il prefisso email-. Impostando questo valore, il server può essere specificamente selezionato per una campagna. Ad esempio: email-primary-server. Caratteri alfanumerici/linea.",
"settings.mailserver.password": "Credenziali (Password)",
"settings.mailserver.passwordHelp": "Entra per modificare",
"settings.mailserver.port": "Porta",
Expand Down
2 changes: 1 addition & 1 deletion i18n/jp.json
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,7 @@
"settings.mailserver.maxConns": "最大接続数",
"settings.mailserver.maxConnsHelp": "サーバーへの最大同時接続数。",
"settings.mailserver.name": "名前",
"settings.mailserver.nameHelp": "SMTPサーバーの任意の一意の名前です。これを設定すると、キャンペーンごとにサーバーを特定のものに選択することができます。例:primary-server。英数字とダッシュのみ使用できます",
"settings.mailserver.nameHelp": "SMTPサーバーのオプションの固有名。email-プレフィックスを付ける必要があります。これにより、キャンペーンに特定のサーバーを選択できます。例:email-primary-server。英数字/ダッシュ",
"settings.mailserver.password": "パスワード",
"settings.mailserver.passwordHelp": "エンターで変更",
"settings.mailserver.port": "ポート",
Expand Down
2 changes: 1 addition & 1 deletion i18n/ml.json
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@
"settings.mailserver.maxConns": "പരമാവധി കണക്ഷനുകൾ",
"settings.mailserver.maxConnsHelp": "SMTP സേർവ്വറിലേയ്ക്കുള്ള പരമാവധി സമാന്തര കണക്ഷനുകൾ.",
"settings.mailserver.name": "പേര്",
"settings.mailserver.nameHelp": "SMTP സേവകത്തിന് ഐച്ഛികമായ അനന്യ പേര്. ഈ സര്‍വര്‍ ഒരു പ്രവര്‍ത്തനത്തിന് പ്രത്യേകം തിരഞ്ഞെടുക്കുന്നതിന് സാധാരണയായി അനുവദിക്കുന്നു. ഉദാ: പ്രധാന-സേവര്‍. അക്ഷരം / ഡാഷ് ഉപയോഗപ്പെടുന്നു.",
"settings.mailserver.nameHelp": "SMTP സെർവറിന് ഓപ്ഷണൽ യൂണിക് പേര്. പൂർവ്വപ്രത്യയത്തോടുക്ക് email- എന്ന പ്രത്യയം ഉണ്ടായിരിക്കുകയുണ്ടാക്കുന്നത് ഏറ്റവും നല്ല ക്യാമ്പെയിൻ വഴി സെർവറിനെ തിരഞ്ഞെടുക്കാൻ അനുവദിക്കും. ഉദാ: email-primary-server. അക്ഷരങ്ങളും / ഡാഷുകളും ഉണ്ടായിരിക്കാം.",
"settings.mailserver.password": "രഹസ്യ കോഡ്‌",
"settings.mailserver.passwordHelp": "മാറ്റം വരുത്താൻ എന്റർ കീ അമർത്തുക",
"settings.mailserver.port": "പോർട്ട്",
Expand Down
2 changes: 1 addition & 1 deletion i18n/nl.json
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@
"settings.mailserver.maxConns": "Max. # connecties",
"settings.mailserver.maxConnsHelp": "Maximum # concurrente connecties naar de server.",
"settings.mailserver.name": "Naam",
"settings.mailserver.nameHelp": "Optionele unieke naam voor de SMTP-server. Het instellen hiervan maakt het mogelijk om specifiek voor een campagne een server te selecteren. Bijv .: primary-server. Alfanumeriek / streepje.",
"settings.mailserver.nameHelp": "Optionele unieke naam voor de SMTP-server. Moet de prefix 'email-' hebben. Deze instelling maakt het mogelijk om de server specifiek te selecteren voor een campagne. bv: email-primary-server. Alfanumeriek / streepje.",
"settings.mailserver.password": "Wachtwoord",
"settings.mailserver.passwordHelp": "Enter om te veranderen",
"settings.mailserver.port": "Poort",
Expand Down
1 change: 1 addition & 0 deletions i18n/no.json
Original file line number Diff line number Diff line change
Expand Up @@ -430,6 +430,7 @@
"settings.mailserver.idleTimeoutHelp": "Tid å vente på ny aktivitet på en tilkobling før den lukkes og fjernes fra bassenget (s for sekunder, m for minutter).",
"settings.mailserver.maxConns": "Maks. tilkoblinger",
"settings.mailserver.maxConnsHelp": "Maksimalt antall samtidige tilkoblinger til serveren.",
"settings.mailserver.nameHelp": "Valgfritt unikt navn for SMTP-serveren. Må ha prefikset email-. Dette tillater at serveren kan spesifikt velges for en kampanje. For eksempel: email-primary-server. Alfanumerisk / bindestrek.",
"settings.mailserver.password": "Passord",
"settings.mailserver.passwordHelp": "Skriv inn for å endre",
"settings.mailserver.port": "Port",
Expand Down
Loading

0 comments on commit 2defef3

Please sign in to comment.