aboutsummaryrefslogtreecommitdiff
path: root/src/main/scala/xyz/driver/pdsuidomain/services/rest/RestHelper.scala
diff options
context:
space:
mode:
authorJakob Odersky <jakob@driver.xyz>2017-07-18 17:30:52 -0700
committerJakob Odersky <jakob@driver.xyz>2017-07-18 17:30:52 -0700
commit99ebbb98068324c2c26dd59484acbe9a8b62ae59 (patch)
tree6b29e0e0a2ce0d940e5c1332362ded1696ddcdb9 /src/main/scala/xyz/driver/pdsuidomain/services/rest/RestHelper.scala
parent71f31dfc68cdc158cc0b3d09b02a54c79a569799 (diff)
downloadrest-query-99ebbb98068324c2c26dd59484acbe9a8b62ae59.tar.gz
rest-query-99ebbb98068324c2c26dd59484acbe9a8b62ae59.tar.bz2
rest-query-99ebbb98068324c2c26dd59484acbe9a8b62ae59.zip
Fixes to error message handling
Diffstat (limited to 'src/main/scala/xyz/driver/pdsuidomain/services/rest/RestHelper.scala')
-rw-r--r--src/main/scala/xyz/driver/pdsuidomain/services/rest/RestHelper.scala8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/main/scala/xyz/driver/pdsuidomain/services/rest/RestHelper.scala b/src/main/scala/xyz/driver/pdsuidomain/services/rest/RestHelper.scala
index 5284ff1..bc886ba 100644
--- a/src/main/scala/xyz/driver/pdsuidomain/services/rest/RestHelper.scala
+++ b/src/main/scala/xyz/driver/pdsuidomain/services/rest/RestHelper.scala
@@ -92,10 +92,10 @@ trait RestHelper {
def extractErrorMessage(response: HttpResponse): Future[String] = {
import xyz.driver.pdsuicommon.serialization.PlayJsonSupport._
Unmarshal(response.entity)
- .to[ErrorsResponse.ResponseError]
+ .to[ErrorsResponse]
.transform(
- _.message,
- ex => new DomainException(ex.getMessage)
+ response => response.errors.map(_.message).mkString(", "),
+ ex => new DomainException("Response has invalid format", ex)
)
}
@@ -108,7 +108,7 @@ trait RestHelper {
case StatusCodes.Forbidden => new AuthorizationException(message)
case StatusCodes.NotFound => new NotFoundException(message)
case other =>
- new DomainException(s"Unhandled domain error for HTTP status ${other.value}. Message ${message}")
+ new DomainException(s"Unhandled domain error for HTTP status ${other.value}. ${message}")
})
}
}