From e43cee77434ffe3012db2c535c8137fbf147c612 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=A0est=C3=A1k=20V=C3=ADt?= Date: Fri, 6 Oct 2017 11:05:57 +0200 Subject: [PATCH] E-mail export: More descriptive error message when some vulnerability is missing --- app/services/EmailExportService.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/services/EmailExportService.scala b/app/services/EmailExportService.scala index 9a9b9ab..5fa17e3 100644 --- a/app/services/EmailExportService.scala +++ b/app/services/EmailExportService.scala @@ -79,7 +79,7 @@ class EmailExportService(from: String, nobodyInterestedContact: String, val expo def emailDigest(subscriber: LoginInfo, changes: Seq[Change], projects: ProjectsWithReports): Future[Email] = { val vulnNames = changes.map(_.vulnerabilityName).toSet for { - vulns <- Future.traverse(vulnNames.toSeq)(name => odcService.getVulnerabilityDetails(name).map(v => name -> v.get)).map(_.toMap) + vulns <- Future.traverse(vulnNames.toSeq)(name => odcService.getVulnerabilityDetails(name).map(v => name -> v.getOrElse(throw new NoSuchElementException(s"Vulnerability details for '$name' not found.")))).map(_.toMap) groups = changes.groupBy(_.direction).withDefaultValue(Seq()) } yield { val changesMarks = Map(Direction.Added -> "❢", Direction.Removed -> "☑")