Skip to content

Commit b566618

Browse files
committed
update again
1 parent 32f9528 commit b566618

2 files changed

Lines changed: 3 additions & 5 deletions

File tree

common/src/main/scala/models/Registration.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,11 +23,11 @@ object Registration {
2323
if (v.validate[Topic].isError) {
2424
val topicType = (v \ "type").asOpt[String].getOrElse("unknown")
2525
val topicName = (v \ "name").asOpt[String].getOrElse("unknown")
26-
Some(s"""$topicType is not a valid topic type (topic name="$topicName")""")
26+
Some(s"Topic Type=$topicType (Topic Id=$topicName)")
2727
} else None
2828
}
2929
if (invalid.nonEmpty)
30-
logger.warn(s"Bad request: ${invalid.mkString(", ")}")
30+
logger.warn(s"Bad request: request contains invalid topic type(s). ${invalid.mkString(". ")}")
3131
}
3232
base.reads(json)
3333
},

registration/app/registration/CustomErrorHandler.scala

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,7 @@ class CustomErrorHandler(env: Environment, config: Configuration, sourceMapper:
1414
private val logger: Logger = LoggerFactory.getLogger(this.getClass)
1515
override def onBadRequest(request: RequestHeader, message: String): Future[Result] = {
1616
val debugInfo = request.headers.get("User-Agent")
17-
if (!message.startsWith("Json validation error")) {
18-
logger.error(s"Bad request due to $message. User agent = $debugInfo")
19-
}
17+
logger.error(s"Bad request due to $message. User agent = $debugInfo")
2018
Future.successful(BadRequest("Bad request"))
2119
}
2220
}

0 commit comments

Comments
 (0)