Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modifiche all'invio delle email: mittente e reply_to #1255

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ToX82
Copy link
Contributor

@ToX82 ToX82 commented Mar 21, 2023

Descrizione

Con questa PR si aggiungono due piccole funzionalità all'invio delle email, entrambi gestibili da file di configurazione:

  • $force_reply_to_sender (false | true). Se true ogni email partirà con il reply_to impostato all'indirizzo email dell'account che sta inviando la mail
  • $force_mail_from_sender (false | true). Se true, per ogni invio email viene controllato se esiste un account SMTP configurato con la stessa email dell'account che sta inviando la mail. Se esiste verrà usato quell'SMTP, altrimenti verrà usato l'account SMTP di default.

Le nuove variabili sono illustrate in config.example.com. Il default è false in entrambi i casi, per mantenere il comportamento attuale, anche qualora le variabili non fossero impostate nel file di configurazione.

Tipologia

Rimuovi le opzioni non rilevanti.

  • Nuova funzionalità (cambiamenti minori che aggiungono una nuova funzionalità)
  • Questo cambiamenti richiede un aggiornamento della documentazione

Checklist

  • Il codice segue le linee guida del progetto
  • Ho commentato il codice, in particolare nelle parti più complesse
  • Il codice non genera warnings

@sajotrei
Copy link

risolverebbe questo punto
Cambio mittente in fase di invio documento #1019

Copy link
Contributor

@lucasalva87 lucasalva87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ottima idea però secondo me c'è un problema alla base con il force_mail_from_sender, ovvero non è detto che il mittente che viene forzato sia supportato dal server SMTP. Es. server SMTP legato al dominio "miosito.it" e il mittente è "@altrosito.it". Inoltre potrebbero esserci problemi di deliverability ("consegnabilità") dell'email, mancanza di record DNS e quindi autorizzazioni SPF,DKIM e DMARC sul dominio forzato che "abilitano" il server STMP ad inviare email anche per conto di quel dominio.

@ToX82
Copy link
Contributor Author

ToX82 commented Apr 11, 2023

Credo che il problema non si ponga nemmeno in quel caso, perché anche se il dominio è diverso, per ogni casella email sono impostati i dati corretti dell'SMTP e le relative credenziali.

@lucasalva87
Copy link
Contributor

lucasalva87 commented Apr 11, 2023

Non è detto che l'email che usi quando attivi il force_mail_from_sender abbia il relativo account smtp configurato a gestionale. Se force_mail_from_sender è a true, prima di forzare il mittente, bisognerebbe allora controllare che esista un account smtp configurato a suo nome o in alternativa ignorare il cambio del mittente (e questo mi sembra che lo faccia già da come scrivevi) corretto?

@ToX82
Copy link
Contributor Author

ToX82 commented Apr 11, 2023

Esattamente. C'è un controllo che verifica se la mail è configurata come account. Se lo è viene utilizzata quella, altrimenti viene utilizzato l'account di default

@lucasalva87
Copy link
Contributor

Perfetto se @loviuz non ha particolari ulteriori osservazioni io direi che gestita così va bene.

@lucasalva87
Copy link
Contributor

Al limite prevedere per il modulo account email possibilità di definire altri mittenti autorizzati per quel server smtp

immagine

@sajotrei
Copy link

Ciao, c'è una previsione per introdurre questo PR in relase ?

@Pek5892
Copy link
Contributor

Pek5892 commented Dec 9, 2024

Questa miglioria era stata rivista e introdotta da @MatteoPistorello, possiamo chiudere la pr?
@lucasalva87 @loviuz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants