Skip to content

Commit c45c94d

Browse files
authored
Merge pull request #152 from ing-bank/feature/stsRequestTime
add sts request time to log
2 parents 0d7989b + 62c7dca commit c45c94d

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/main/scala/com/ing/wbaa/rokku/proxy/api/ProxyService.scala

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,10 @@ trait ProxyService {
6868
withoutSizeLimit {
6969
extractRequest { httpRequest =>
7070
extracts3Request { s3Request =>
71+
val requestStartTime = System.nanoTime()
7172
onComplete(areCredentialsActive(s3Request.credential)) {
7273
case Success(Some(userSTS: User)) =>
73-
logger.info("STS credentials active for request, user retrieved: {}", userSTS)
74+
logger.info("STS credentials active for request, user retrieved: {} (response time {})", userSTS, System.nanoTime() - requestStartTime)
7475
onComplete(processRequestForValidUser(httpRequest, s3Request, userSTS)) {
7576
case Success(r) => r
7677
case Failure(exception) =>
@@ -80,11 +81,11 @@ trait ProxyService {
8081
}
8182
case Success(None) =>
8283
implicit val returnStatusCode: StatusCodes.ClientError = StatusCodes.Forbidden
83-
logger.warn("STS credentials not active: {}", s3Request)
84+
logger.warn("STS credentials not active: {} (response time {})", s3Request, System.nanoTime() - requestStartTime)
8485
complete(returnStatusCode -> AwsErrorCodes.response(returnStatusCode))
8586
case Failure(exception) =>
8687
implicit val returnStatusCode: StatusCodes.ServerError = StatusCodes.InternalServerError
87-
logger.error("An error occurred when checking credentials with STS service ex={}", exception)
88+
logger.error("An error occurred when checking credentials with STS service ex={} (response time {})", exception, System.nanoTime() - requestStartTime)
8889
complete(returnStatusCode -> AwsErrorCodes.response(returnStatusCode))
8990
}
9091
}

0 commit comments

Comments
 (0)