chore: drop restart application on error (#263)

navikt · Oct 15, 2024 · c535421 · c535421
1 parent 182471b
commit c535421
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 13 deletions.
diff --git a/src/main/kotlin/no/nav/pdfgen/Application.kt b/src/main/kotlin/no/nav/pdfgen/Application.kt
@@ -1,7 +1,5 @@
 package no.nav.pdfgen
 
-// Uncommemt to enable debug to file
-// import java.io.File
 import com.openhtmltopdf.slf4j.Slf4jLogger
 import com.openhtmltopdf.util.XRLog
 import io.ktor.server.application.*
@@ -50,10 +48,10 @@ fun Application.module() {
     val templates = coreEnvironment.templates
     XRLog.setLoggerImpl(Slf4jLogger())
 
-    configureLifecycleHooks(applicationState)
+    configureLifecycleHooks(applicationState = applicationState)
     configureContentNegotiation()
-    configureStatusPages(templates, applicationState)
-    configureNais(applicationState)
+    configureStatusPages(templates = templates)
+    configureNais(applicationState = applicationState)
     configureReloadPDFGenCore(environment = environment)
     configureRouting(environment = environment)
 }

diff --git a/src/main/kotlin/no/nav/pdfgen/plugins/LifecycleHooks.kt b/src/main/kotlin/no/nav/pdfgen/plugins/LifecycleHooks.kt
@@ -5,6 +5,11 @@ import no.nav.pdfgen.ApplicationState
 
 fun Application.configureLifecycleHooks(applicationState: ApplicationState) {
 
-    this.monitor.subscribe(ApplicationStarted) { applicationState.ready = true }
-    this.monitor.subscribe(ApplicationStopped) { applicationState.ready = false }
+    monitor.subscribe(ApplicationStarted) {
+        applicationState.ready = true
+    }
+    monitor.subscribe(ApplicationStopped) {
+        applicationState.ready = false
+        applicationState.alive = false
+    }
 }
diff --git a/src/main/kotlin/no/nav/pdfgen/plugins/StatusPages.kt b/src/main/kotlin/no/nav/pdfgen/plugins/StatusPages.kt
@@ -12,14 +12,11 @@ import no.nav.pdfgen.logger
 
 fun Application.configureStatusPages(
     templates: Map<Pair<String, String>, Template>,
-    applicationState: ApplicationState
 ) {
     install(StatusPages) {
         exception<Throwable> { call, cause ->
-            call.respond(HttpStatusCode.InternalServerError, cause.message ?: "Unknown error")
             logger.error("Caught exception", cause)
-            applicationState.alive = false
-            applicationState.ready = false
+            call.respond(HttpStatusCode.InternalServerError, cause.message ?: "Unknown error")
         }
         status(HttpStatusCode.NotFound) { call, _ ->
             call.respond(

diff --git a/src/main/resources/logback.xml b/src/main/resources/logback.xml
@@ -7,12 +7,11 @@
     </appender>
 
     <logger name="pdfgen" level="INFO"/>
+    <logger name="org.apache.pdfbox" level="ERROR" />
 
     <root level="INFO">
         <appender-ref ref="stdout_json" />
     </root>
 
-    <logger name="org.apache.pdfbox" level="ERROR" />
-
 </configuration>