Build: Set env varibales for pre-build step (Github Workflow)

jknack · jknack · commit 6fb133e68e33 · 2020-01-08T15:17:21.000-03:00
diff --git a/.github/workflows/full-build.yml b/.github/workflows/full-build.yml
@@ -19,7 +19,10 @@ jobs:
         with:
           java-version: ${{ matrix.java_version }}
       - name: Install
-        run: mvn clean install --fail-never -q -pl '!docs' -P checkstyle
+        run: mvn clean install --fail-never -q -pl '!docs'
+        env:
+          BUILD_PORT: 0
+          BUILD_SECURE_PORT: 0
       - name: Build
         run: mvn clean checkstyle:checkstyle package -pl '!docs' -P checkstyle
         env:
diff --git a/.github/workflows/quick-build.yml b/.github/workflows/quick-build.yml
@@ -20,7 +20,10 @@ jobs:
         with:
           java-version: ${{ matrix.java_version }}
       - name: Install
-        run: mvn clean install --fail-never -q -pl '!docs' -P checkstyle
+        run: mvn clean install --fail-never -q -pl '!docs'
+        env:
+          BUILD_PORT: 0
+          BUILD_SECURE_PORT: 0
       - name: Build
         run: mvn clean checkstyle:checkstyle package -pl '!docs' -P checkstyle
         env:
diff --git a/jooby/src/main/java/io/jooby/DefaultContext.java b/jooby/src/main/java/io/jooby/DefaultContext.java
@@ -571,8 +571,13 @@ default Context setResponseHeader(@Nonnull String name, @Nonnull Object value) {
         }
         router.getErrorHandler().apply(this, cause, statusCode);
       } catch (Exception x) {
-        router.getLog()
-            .error("error handler resulted in exception {} {}", getMethod(), pathString(), x);
+        if (Server.connectionLost(x)) {
+          router.getLog()
+              .debug("error handler resulted in exception {} {}", getMethod(), getRequestPath(), x);
+        } else {
+          router.getLog()
+              .error("error handler resulted in exception {} {}", getMethod(), getRequestPath(), x);
+        }
       }
     }
     /** rethrow fatal exceptions: */

Original file line number	Diff line number	Diff line change
`@@ -571,8 +571,13 @@ default Context setResponseHeader(@Nonnull String name, @Nonnull Object value) {`
`571`	`571`	`}`
`572`	`572`	`router.getErrorHandler().apply(this, cause, statusCode);`
`573`	`573`	`} catch (Exception x) {`
`574`		`- router.getLog()`
`575`		`- .error("error handler resulted in exception {} {}", getMethod(), pathString(), x);`
	`574`	`+ if (Server.connectionLost(x)) {`
	`575`	`+ router.getLog()`
	`576`	`+ .debug("error handler resulted in exception {} {}", getMethod(), getRequestPath(), x);`
	`577`	`+ } else {`
	`578`	`+ router.getLog()`
	`579`	`+ .error("error handler resulted in exception {} {}", getMethod(), getRequestPath(), x);`
	`580`	`+ }`
`576`	`581`	`}`
`577`	`582`	`}`
`578`	`583`	`/** rethrow fatal exceptions: */`