Skip to content

Commit 59f32bf

Browse files
committed
Update Jetty to 9.4
- from 9.0.3.v20130506 to 9.4.52.v20230823 - rearrange and reorganize servlets and handlers so that the server provides everything - remove server session attribute - we already handle user sessions separately at the moment - mark services known to be unused as deprecated - add plugin controller check to skip JettyPluginInterface resource if it is null
1 parent c846c09 commit 59f32bf

File tree

7 files changed

+159
-128
lines changed

7 files changed

+159
-128
lines changed

dicoogle/src/main/java/pt/ua/dicoogle/plugins/PluginController.java

+3
Original file line numberDiff line numberDiff line change
@@ -256,6 +256,9 @@ private void initJettyInterface(Collection<PluginSet> plugins) {
256256

257257
DicoogleWeb jettyServer = ControlServices.getInstance().getWebServicePlatform();
258258
for (JettyPluginInterface resource : jettyInterfaces) {
259+
if (resource == null) {
260+
continue;
261+
}
259262
jettyServer.addContextHandlers(resource.getJettyHandlers());
260263
}
261264
}

dicoogle/src/main/java/pt/ua/dicoogle/server/web/DicoogleWeb.java

+147-124
Large diffs are not rendered by default.

dicoogle/src/main/java/pt/ua/dicoogle/server/web/auth/Session.java

-3
Original file line numberDiff line numberDiff line change
@@ -235,9 +235,6 @@ public static LoggedInStatus webappLogin(HttpServletRequest request, HttpServlet
235235
return new LoggedInStatus(null, LoggedInStatus.S_UNAUTHORIZEDACCESS);
236236
}
237237

238-
// add the login information to the session
239-
HttpSession session = request.getSession(true); // force the creation of a new session if there is none
240-
session.setAttribute("login", login);
241238
return new LoggedInStatus(login, LoggedInStatus.S_VALIDLOGIN);
242239
}
243240

dicoogle/src/main/java/pt/ua/dicoogle/server/web/servlets/SearchHolderServlet.java

+2
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,9 @@
4343
*
4444
* @author Tiago Godinho.
4545
*
46+
* @deprecated See {@link pt.ua.dicoogle.server.web.servlets.search.SearchServlet} instead
4647
*/
48+
@Deprecated
4749
public class SearchHolderServlet extends HttpServlet {
4850
private static final long serialVersionUID = 1L;
4951

dicoogle/src/main/java/pt/ua/dicoogle/server/web/servlets/search/ExportServlet.java

+4
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,10 @@
3535
import pt.ua.dicoogle.plugins.PluginController;
3636
import pt.ua.dicoogle.sdk.utils.DictionaryAccess;
3737

38+
/**
39+
* @deprecated Use {@link pt.ua.dicoogle.server.web.servlets.ExportCSVToFILEServlet} instead.
40+
*/
41+
@Deprecated
3842
public class ExportServlet extends HttpServlet {
3943
private static final Logger logger = LoggerFactory.getLogger(ExportServlet.class);
4044

dicoogle/src/main/java/pt/ua/dicoogle/server/web/servlets/search/WadoServlet.java

+2
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,9 @@
3838
/**
3939
* @author Frederico Silva <[email protected]>
4040
* @todo
41+
* @deprecated Incomplete implementation, to be removed in Dicoogle 4
4142
*/
43+
@Deprecated
4244
public class WadoServlet extends HttpServlet {
4345

4446
@Override

pom.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
<name>dicoogle-all</name>
99

1010
<properties>
11-
<jetty.version>9.0.3.v20130506</jetty.version>
11+
<jetty.version>9.4.52.v20230823</jetty.version>
1212
<restlet.version>2.1.2</restlet.version>
1313
<dcm4che.version>2.0.29</dcm4che.version>
1414
<slf4j.version>1.7.36</slf4j.version>

0 commit comments

Comments
 (0)