Skip to content
Permalink
Browse files

[FIXED JENKINS-46659] Avoid Jetty timeouts by sending a ping-like pac…

…ket every 10s while a CLI command is running.

(cherry picked from commit dc06c3f)
  • Loading branch information...
jglick authored and olivergondza committed Sep 9, 2017
1 parent 06b0cd6 commit dda5903c99a46df0d70b7d143d4fd14389bd433b
Showing with 16 additions and 8 deletions.
  1. +16 −0 cli/src/main/java/hudson/cli/CLI.java
  2. +0 −8 cli/src/main/java/hudson/cli/PlainCLIProtocol.java
@@ -704,6 +704,22 @@ public void run() {
}
}
}.start();
new Thread("ping") { // JENKINS-46659
@Override
public void run() {
try {
Thread.sleep(10_000);
while (!connection.complete) {
LOGGER.fine("sending ping");
connection.sendEncoding(Charset.defaultCharset().name()); // no-op at this point
Thread.sleep(10_000);
}
} catch (IOException | InterruptedException x) {
LOGGER.log(Level.WARNING, null, x);
}
}

}.start();
synchronized (connection) {
while (!connection.complete) {
connection.wait();
@@ -34,7 +34,6 @@
import java.io.OutputStream;
import java.nio.channels.ClosedChannelException;
import java.nio.channels.ReadPendingException;
import java.util.concurrent.TimeoutException;
import java.util.logging.Level;
import java.util.logging.Logger;
import org.apache.commons.io.IOUtils;
@@ -112,13 +111,6 @@ public void run() {
} catch (EOFException x) {
handleClose();
break; // TODO verify that we hit EOF immediately, not partway into framelen
} catch (IOException x) {
if (x.getCause() instanceof TimeoutException) { // TODO on Tomcat this seems to be SocketTimeoutException
LOGGER.log(Level.FINE, "ignoring idle timeout, perhaps from Jetty", x);
continue;
} else {
throw x;
}
}
if (framelen < 0) {
throw new IOException("corrupt stream: negative frame length");

0 comments on commit dda5903

Please sign in to comment.
You can’t perform that action at this time.