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

* Your Email Address
* Spam protection	Enter the letters below: . .. ,.__ .___.__ \ / \./ [__)[__ [__) \/ \| [__)[___[__)
Comment	>> (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@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.
Attachment
Watch this ticket