[ruby/uri] lib/uri/mailto.rb (EMAIL_REGEXP): use assertions surrounding the local part instead of a character class

ruby/uri@2d7d2d9988

Author: nlevchuk

lib/uri/mailto.rb

@@ -52,7 +52,7 @@ class MailTo < Generic
     HEADER_REGEXP  = /\A(?<hfield>(?:%\h\h|[!$'-.0-;@-Z_a-z~])*=(?:%\h\h|[!$'-.0-;@-Z_a-z~])*)(?:&\g<hfield>)*\z/
     # practical regexp for email address
     # https://html.spec.whatwg.org/multipage/input.html#valid-e-mail-address
-    EMAIL_REGEXP = /\A[^.][a-zA-Z0-9.!\#$%&'*+\/=?^_`{|}~-]+[^.]@[a-zA-Z0-9](?:[a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?(?:\.[a-zA-Z0-9](?:[a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?)*\z/
+    EMAIL_REGEXP = /\A(?!\.)[a-zA-Z0-9.!\#$%&'*+\/=?^_`{|}~-]+(?<!\.)@[a-zA-Z0-9](?:[a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?(?:\.[a-zA-Z0-9](?:[a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?)*\z/
     # :startdoc:
 
     #

test/uri/test_mailto.rb

@@ -141,6 +141,11 @@ def test_initializer
   def test_check_to
     u = URI::MailTo.build(['[email protected]', 'subject=Ruby'])
 
+    # Valid emails
+    u.to = '[email protected]'
+    assert_equal(u.to, '[email protected]')
+
+    # Invalid emails
     assert_raise(URI::InvalidComponentError) do
       u.to = '#[email protected]'
     end
@@ -156,6 +161,10 @@ def test_check_to
     assert_raise(URI::InvalidComponentError) do
       u.to = '[email protected]'
     end
+
+    assert_raise(URI::InvalidComponentError) do
+      u.to = '[email protected]'
+    end
   end
 
   def test_to_s