Wrap request so we don't NPE when it is null

guardian · Mar 31, 2017 · 37f0d08 · 37f0d08
1 parent d17f603
commit 37f0d08
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/magenta-lib/src/main/scala/magenta/tasks/AWS.scala b/magenta-lib/src/main/scala/magenta/tasks/AWS.scala
@@ -470,9 +470,9 @@ trait AWS extends Loggable {
     override def shouldRetry(originalRequest: AmazonWebServiceRequest, exception: AmazonClientException, retriesAttempted: Int): Boolean = {
       val willRetry = super.shouldRetry(originalRequest, exception, retriesAttempted)
       if (willRetry) {
-        logger.warn(s"AWS SDK retry $retriesAttempted: ${originalRequest.getClass.getName} threw ${exceptionInfo(exception)}")
+        logger.warn(s"AWS SDK retry $retriesAttempted: ${Option(originalRequest).map(_.getClass.getName} threw ${exceptionInfo(exception)}")
       } else {
-        logger.warn(s"Encounted fatal exception during AWS API call", exception)
+        logger.warn(s"Encountered fatal exception during AWS API call", exception)
         Option(exception.getCause).foreach(t => logger.warn(s"Cause of fatal exception", t))
       }
       willRetry