After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 713355 - Problems validating email addresses in composer
Problems validating email addresses in composer
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: composer
unspecified
Other All
: Normal normal
: ---
Assigned To: Geary Maintainers
Geary Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-04-12 06:19 UTC by Jim Nelson
Modified: 2018-01-11 00:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Charles Lindsay 2013-11-21 20:20:06 UTC 


---- Reported by jim@yorba.org 2012-04-12 11:19:00 -0700 ----

Original Redmine bug id: 5047
Original URL: http://redmine.yorba.org/issues/5047
Searchable id: yorba-bug-5047
Original author: Jim Nelson
Original description:

There are two corner cases when the composer validates email addresses:

  * This is considered valid: "Jim Nelson <" 
  * This is considered valid: "Jim Nelson <jim@yorba.org"

The first one is obviously problematic. The second may be accepted by some
mailers.

Related issues:
related to geary - 4313: Mail composer: validate e-mail addresses (Fixed)
related to geary - 5904: Sender with Last Name, First causes parse
problem in comp... (Open)



---- Additional Comments From geary-maint@gnome.bugs 2013-01-14 17:39:00 -0800 ----

### History

####

#1

Updated by Jim Nelson over 1 year ago

  * **Description** updated (diff)
  * **Category** set to _13_
  * **Priority** changed from _Normal_ to _High_
  * **Target version** set to _0.1_

####

#2

Updated by Adam Dingle over 1 year ago

  * **Target version** deleted (<strike>_0.1_</strike>)

####

#3

Updated by Jim Nelson 10 months ago

  * **Category** set to _composer_
  * **Priority** changed from _High_ to _Normal_



--- Bug imported by chaz@yorba.org 2013-11-21 20:20 UTC  ---

This bug was previously known as _bug_ 5047 at http://redmine.yorba.org/show_bug.cgi?id=5047

Unknown version " in product geary. 
   Setting version to "!unspecified".
Unknown milestone "unknown in product geary. 
   Setting to default milestone for this product, "---".
Setting qa contact to the default for this product.
   This bug either had no qa contact or an invalid one.
Resolution set on an open status.
   Dropping resolution
Comment 1 Federico Bruni 2018-01-09 13:25:48 UTC 
I'm trying on current Geary 0.13-dev.
This is no more considered valid (Send button is not active):

"Jim Nelson <"

So far so good.

While this is considered valid:

"Jim Nelson <jim@yorba.or"

In other words, it validates as soon as it matches a domain of at least two characters. I don't think we can do any better.

And the missing closing > is not a problem: I've just sent an email to an address of mine written as "Me <me@domain.com" and it arrived correctly.
But it would be nice if Geary autoclosed it (added the missing >) when the user changes field.
Comment 2 Michael Gratton 2018-01-11 00:45:19 UTC 
Cool, glad the validation part is fixed. I've filed Bug 792418 to tidy this up and take care of the normalisation part.