diff --git a/apps/jetty/build.xml b/apps/jetty/build.xml index 0377e74b89..f3de9bffa0 100644 --- a/apps/jetty/build.xml +++ b/apps/jetty/build.xml @@ -262,9 +262,6 @@ - diff --git a/installer/resources/eepsite/jetty-jmx.xml b/installer/resources/eepsite/jetty-jmx.xml index 4db0dbb473..aca96f7c2c 100644 --- a/installer/resources/eepsite/jetty-jmx.xml +++ b/installer/resources/eepsite/jetty-jmx.xml @@ -1,103 +1,43 @@ - + - - - - - - - - + - + - + - - - - - - - - - - - - - - - true + + + + + + + - - - - - - - - - - - - - - - + diff --git a/installer/resources/eepsite/jetty-rewrite.xml b/installer/resources/eepsite/jetty-rewrite.xml index 1d208f8bfc..1f93937d5b 100644 --- a/installer/resources/eepsite/jetty-rewrite.xml +++ b/installer/resources/eepsite/jetty-rewrite.xml @@ -1,5 +1,5 @@ - + @@ -9,13 +9,13 @@ - + - + true false requestedPath @@ -68,7 +68,7 @@ - + @@ -89,7 +89,7 @@ - + @@ -113,5 +113,5 @@ - + diff --git a/installer/resources/eepsite/jetty-ssl.xml b/installer/resources/eepsite/jetty-ssl.xml index 79fe341f88..5b42684334 100644 --- a/installer/resources/eepsite/jetty-ssl.xml +++ b/installer/resources/eepsite/jetty-ssl.xml @@ -1,6 +1,8 @@ + + diff --git a/installer/resources/eepsite/jetty.xml b/installer/resources/eepsite/jetty.xml index 51784fb873..dd2be4dddb 100644 --- a/installer/resources/eepsite/jetty.xml +++ b/installer/resources/eepsite/jetty.xml @@ -1,6 +1,8 @@ + + diff --git a/router/java/src/net/i2p/router/startup/MigrateJetty.java b/router/java/src/net/i2p/router/startup/MigrateJetty.java index c4ee2b65b6..db6476baac 100644 --- a/router/java/src/net/i2p/router/startup/MigrateJetty.java +++ b/router/java/src/net/i2p/router/startup/MigrateJetty.java @@ -215,6 +215,12 @@ abstract class MigrateJetty { * @since Jetty 9 */ private static boolean migrateToJetty9(File xmlFile) { + if (xmlFile.getName().equals("jetty-jmx.xml")) { + // This is lazy but nobody's using jmx, not worth the trouble + System.err.println("ERROR: Migration of " + xmlFile + + " file is not supported. Copy new file from $I2P/eepsite-jetty9/jetty-jmx.xml"); + return false; + } // we don't re-migrate from the template, we just add the // necessary args for the QueuedThreadPool constructor in-place // and fixup the renamed set call @@ -237,7 +243,13 @@ abstract class MigrateJetty { // readLine() doesn't strip \r if (s.endsWith("\r")) s = s.substring(0, s.length() - 1); - if (s.contains("org.eclipse.jetty.util.thread.QueuedThreadPool")) { + if (s.contains("Modified by I2P migration script for Jetty 9.") || + s.contains("This configuration supports Jetty 9.") || + s.contains("http://www.eclipse.org/jetty/configure_9_0.dtd")) { + if (!modified) + break; + // else we've modified it twice? + } else if (s.contains("org.eclipse.jetty.util.thread.QueuedThreadPool")) { foundQTP = true; } else if (foundQTP) { if (!(s.contains("Modified by") || s.contains(""))) {