From d930ec04ff22b492980c813577daed2fe282138c Mon Sep 17 00:00:00 2001 From: vkumra-broad Date: Wed, 30 May 2018 16:30:14 -0400 Subject: [PATCH] Date accessed should only be updated when cluster is ready (#396) --- .../dsde/workbench/leonardo/service/ProxyService.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/service/ProxyService.scala b/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/service/ProxyService.scala index b39b35277b7..fbb94af7fa6 100644 --- a/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/service/ProxyService.scala +++ b/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/service/ProxyService.scala @@ -122,9 +122,9 @@ class ProxyService(proxyConfig: ProxyConfig, private def proxyInternal(userInfo: UserInfo, googleProject: GoogleProject, clusterName: ClusterName, request: HttpRequest): Future[HttpResponse] = { logger.debug(s"Received proxy request for user user $userInfo") - clusterDateAccessedActor ! UpdateDateAccessed(clusterName, googleProject, Instant.now()) getTargetHost(googleProject, clusterName) flatMap { case ClusterReady(targetHost) => + clusterDateAccessedActor ! UpdateDateAccessed(clusterName, googleProject, Instant.now()) // If this is a WebSocket request (e.g. wss://leo:8080/...) then akka-http injects a // virtual UpgradeToWebSocket header which contains facilities to handle the WebSocket data. // The presence of this header distinguishes WebSocket from http requests.