antville/helma
antville/helma
1
1
Fork 0
Code Issues 5 Pull requests 7 Projects Releases 5 Activity Actions

Update Jetty packages (major) #103

Open
renovate[bot] wants to merge 46 commits from renovate/major-jetty-packages into main
pull from: renovate/major-jetty-packages
merge into: antville:main
Conversation 4 Commits 46 Files changed 15 +4 -4
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 2994a4becc - Show all commits

8
src/main/java/helma/main/ApplicationManager.java
View file

@ -502,7 +502,9 @@ public class ApplicationManager implements XmlRpcHandler {
staticContext.start();
}
appContext = new ServletContextHandler(ServletContextHandler.SESSIONS);
// I hope I am correct assuming Helma does not need Jetty’s session management, but using
// `ServletContextHandler.SESSIONS` causes an exception: Shared scheduler not started
appContext = new ServletContextHandler(ServletContextHandler.NO_SESSIONS);
appContext.setContextPath(pathPattern);
context.addHandler(appContext);
@ -544,9 +546,7 @@ public class ApplicationManager implements XmlRpcHandler {
// Remap the context paths and start
ApplicationManager.this.context.mapContexts();
// FIXME: Causing java.lang.IllegalStateException: Shared scheduler not started
// Is it necessary, anway?
//this.appContext.start();
this.appContext.start();
}
// register as XML-RPC handler