Skip to content

Commit 2840938

Browse files
authored
Merge pull request #79 from scala-exercises/enrique-0-5-0-memory-fix
Memory leak bug fix
2 parents d566f4e + aed7411 commit 2840938

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

server/src/main/scala/org/scalaexercises/evaluator/services.scala

+3-4
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,6 @@ object services {
2828

2929
import EvalResponse.messages._
3030

31-
def evaluatorInstance[F[_]: ConcurrentEffect: ContextShift: Timer: Sync] =
32-
new Evaluator[F](20 seconds)
33-
3431
val corsHeaders = Seq(
3532
Header("Vary", "Origin,Access-Control-Request-Methods"),
3633
Header("Access-Control-Allow-Methods", "POST"),
@@ -116,12 +113,14 @@ object EvaluatorServer extends IOApp {
116113
lazy val port = (Option(System.getenv("PORT")) orElse
117114
Option(System.getProperty("http.port"))).map(_.toInt).getOrElse(8080)
118115

116+
lazy val evaluator = new Evaluator[IO](15 seconds)
117+
119118
override def run(args: List[String]): IO[ExitCode] = {
120119
logger.info(s"Initializing Evaluator at $ip:$port")
121120

122121
BlazeServerBuilder[IO]
123122
.bindHttp(port, ip)
124-
.withHttpApp(auth[IO](service(evaluatorInstance)))
123+
.withHttpApp(auth[IO](service(evaluator)))
125124
.serve
126125
.compile
127126
.lastOrError

0 commit comments

Comments
 (0)