[Tickets #14247] Re: Check email address for event reminders

[noreply at bugs.horde.org]

BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE  
E-MAIL-ADRESSE WERDEN NICHT GELESEN.

Ticket-URL: https://bugs.horde.org/ticket/14247
------------------------------------------------------------------------------
  Ticket           | 14247
  Aktualisiert Von | Jan Schneider <jan at horde.org>
  Zusammenfassung  | Check email address for event reminders
  Warteschlange    | Kronolith
  Version          | FRAMEWORK_5_2
  Typ              | Enhancement
-Status           | New
+Status           | Feedback
  Priorität        | 1. Low
  Milestone        |
  Patch            |
  Zuständige       |
------------------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2016-02-12 12:01) hat geschrieben:

> (1) Append the default domain to unqualified addresses just like IMP  
> does. This may be a bit difficult to do since the domains that would  
> need to be appended are only set in IMP's backends.local.php, so  
> that info might not be available to Kronolith. Maybe the form should  
> be re-displayed in this case with a note to the user to make clear  
> that the email address has been changed automatically.

Not an option, because of the reasons that you mentioned.

> (2) Do at least some basic checking of user input, i.e. validate  
> that it is a syntactically correct email address.  For instance, an  
> input such as "a at b@c" should never be accepted.

No reason to do just basic checking. The only question is whether to  
use Horde_Mail_Rfc822#parseAddressList() or  
Horde_Form_Type_email#validateEmailAddress() with optional SMTP server  
checking.