add missing execution context · devhttps/frontend@2aa1f40 · GitHub

devhttps / frontend Public

forked from guardian/frontend

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Wiki
• Security and quality
• Insights

Additional navigation options

Commit 2aa1f40

Browse filesBrowse the repository at this point in the historyBrowse files

QuarpT

committed

add missing execution context

1 parent bc980f4 commit 2aa1f40
Copy full SHA for 2aa1f40

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• facia/app/controllers/front
  • FrontJsonFapi.scala

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎facia/app/controllers/front/FrontJsonFapi.scala‎

Copy file name to clipboardExpand all lines: facia/app/controllers/front/FrontJsonFapi.scala

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -28,7 +28,7 @@ trait FrontJsonFapi extends Logging {
28	28	pressedPageFromS3(getAddressForPath(path, LiteType.suffix))
29	29	}
30	30
31		- private def parsePressedPage(jsonStringOpt: Option[String]): Future[Option[PressedPage]] = futureSemaphore.execute {
	31	+ private def parsePressedPage(jsonStringOpt: Option[String])(implicit executionContext: ExecutionContext): Future[Option[PressedPage]] = futureSemaphore.execute {
32	32	blockingOperations.executeBlocking {
33	33	jsonStringOpt.map { jsonString =>
34	34	DurationMetric.withMetrics(FaciaPressMetrics.FrontDecodingLatency) {

0 commit comments

Comments

0 (0)