Skip to content

Commit fc7985e

Browse files
authored
Merge pull request #45670 from stuartwdouglas/stuartwdouglas/app-restarts
Don't allow requests into restarting application
2 parents e9d7758 + 4ba3a46 commit fc7985e

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/devmode/VertxHttpHotReplacementSetup.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,9 @@ public Void call() {
135135
routingContext.request().resume();
136136
return;
137137
}
138-
if ((nextUpdate > System.currentTimeMillis() && !hotReplacementContext.isTest())
138+
if ((nextUpdate > System.currentTimeMillis() &&
139+
!hotReplacementContext.isTest() &&
140+
!DevConsoleManager.isDoingHttpInitiatedReload()) // if there is a live reload possibly going on we don't want to let a request through to restarting application, this is best effort, but it narrows the window a lot
139141
|| routingContext.request().headers().contains(HEADER_NAME)) {
140142
if (hotReplacementContext.getDeploymentProblem() != null) {
141143
handleDeploymentProblem(routingContext, hotReplacementContext.getDeploymentProblem());

0 commit comments

Comments
 (0)