Merge pull request #248 from hmrc/Shivangi/DLS-11277_email_address_va…

…lidation_V2

DLS-11277 : Logs added for failed email validation

app/uk/gov/hmrc/cbcr/emailaddress/EmailAddress.scala

@@ -17,24 +17,25 @@
 package uk.gov.hmrc.cbcr.emailaddress
 
 import com.google.inject.ImplementedBy
+import play.api.Logger
 import uk.gov.hmrc.cbcr.emailaddress.EmailAddressValidation.validEmail
 
-import javax.naming.Context.{INITIAL_CONTEXT_FACTORY => ICF}
 import javax.inject.Singleton
+import javax.naming.Context.{INITIAL_CONTEXT_FACTORY => ICF}
 import javax.naming.directory.InitialDirContext
 import scala.jdk.CollectionConverters._
 import scala.util.Try
 import scala.util.matching.Regex
 
 case class EmailAddress(value: String) extends StringValue {
 
+  lazy val obfuscated: ObfuscatedEmailAddress = ObfuscatedEmailAddress.apply(value)
+
   val (mailbox, domain): (Mailbox, Domain) = value match {
     case validEmail(m, d) => (Mailbox(m), Domain(d))
-    case invalidEmail     => throw new IllegalArgumentException(s"'$invalidEmail' is not a valid email address")
+    case invalidEmail =>
+      throw new IllegalArgumentException(s"'$invalidEmail' is not a valid email address")
   }
-
-  lazy val obfuscated: ObfuscatedEmailAddress = ObfuscatedEmailAddress.apply(value)
-
 }
 case class Mailbox(value: String) extends StringValue
 
@@ -46,6 +47,7 @@ case class Domain(value: String) extends StringValue {
 }
 @ImplementedBy(classOf[EmailAddressValidation])
 trait EmailValidation {
+  lazy val logger: Logger = Logger(getClass)
   def isValid(email: String): Boolean
 }
 
@@ -75,7 +77,10 @@ class EmailAddressValidation extends EmailValidation {
   def isValid(email: String): Boolean =
     email match {
       case validEmail(_, _) if isHostMailServer(EmailAddress(email).domain) => true
-      case _                                                                => false
+      case _ =>
+        val obfuscatedEmail = EmailAddress(email).obfuscated
+        logger.error(s"Invalid email Address : $obfuscatedEmail ")
+        false
     }
 }
 

project/plugins.sbt

@@ -3,7 +3,7 @@ resolvers += Resolver.url("HMRC-open-artefacts-ivy", url("https://open.artefacts
   Resolver.ivyStylePatterns)
 resolvers += Resolver.jcenterRepo
 
-addSbtPlugin("uk.gov.hmrc"        % "sbt-auto-build"         % "3.22.0")
+addSbtPlugin("uk.gov.hmrc"        % "sbt-auto-build"         % "3.24.0")
 addSbtPlugin("uk.gov.hmrc"        % "sbt-distributables"     % "2.5.0")
 addSbtPlugin("org.playframework"  %% "sbt-plugin"            % "3.0.5")
 addSbtPlugin("org.scoverage"      % "sbt-scoverage"          % "2.0.12")