Merge pull request #1213 from alphagov/allow-apostrophe

Permit apostrophe in email address
alphagov · Nov 8, 2023 · ba11ac8 · ba11ac8
2 parents 79c060f + 7338e20
commit ba11ac8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/app/models/support/requests/requester.rb b/app/models/support/requests/requester.rb
@@ -7,7 +7,7 @@ class Requester
       attr_accessor :name
       attr_reader :email
 
-      VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
+      VALID_EMAIL_REGEX = /\A[\w+\-.']+@[a-z\d\-.]+\.[a-z]+\z/i
 
       validates :email, presence: true
 

diff --git a/spec/models/support/requests/requester_spec.rb b/spec/models/support/requests/requester_spec.rb
@@ -12,10 +12,12 @@ module Requests
       it { should allow_value("ab @c.com").for(:email) }
       it { should allow_value("[email protected] ").for(:email) }
       it { should allow_value(" [email protected]").for(:email) }
+      it { should allow_value("a'[email protected] ").for(:email) }
       it { should_not allow_value("ab").for(:email) }
 
       it { should allow_value("").for(:collaborator_emails) }
       it { should allow_value("[email protected]").for(:collaborator_emails) }
+      it { should allow_value("a'[email protected]").for(:collaborator_emails) }
       it { should allow_value("[email protected], [email protected]").for(:collaborator_emails) }
       it { should_not allow_value("ab, [email protected]").for(:collaborator_emails) }
       it { should_not allow_value("[email protected]").for(:collaborator_emails) }