Skip to content

Commit

Permalink
Update Jetty to 9.4
Browse files Browse the repository at this point in the history
- 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
  • Loading branch information
Enet4 committed Nov 20, 2024
1 parent 15c65f5 commit cf3735d
Show file tree
Hide file tree
Showing 7 changed files with 159 additions and 128 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -266,6 +266,9 @@ private void initJettyInterface(Collection<PluginSet> plugins) {

DicoogleWeb jettyServer = ControlServices.getInstance().getWebServicePlatform();
for (JettyPluginInterface resource : jettyInterfaces) {
if (resource == null) {
continue;
}
jettyServer.addContextHandlers(resource.getJettyHandlers());
}
}
Expand Down
271 changes: 147 additions & 124 deletions dicoogle/src/main/java/pt/ua/dicoogle/server/web/DicoogleWeb.java

Large diffs are not rendered by default.

Original file line number Diff line number Diff line change
Expand Up @@ -235,9 +235,6 @@ public static LoggedInStatus webappLogin(HttpServletRequest request, HttpServlet
return new LoggedInStatus(null, LoggedInStatus.S_UNAUTHORIZEDACCESS);
}

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

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,9 @@
*
* @author Tiago Godinho.
*
* @deprecated See {@link pt.ua.dicoogle.server.web.servlets.search.SearchServlet} instead
*/
@Deprecated
public class SearchHolderServlet extends HttpServlet {
private static final long serialVersionUID = 1L;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,10 @@
import pt.ua.dicoogle.plugins.PluginController;
import pt.ua.dicoogle.sdk.utils.DictionaryAccess;

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

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,9 @@
/**
* @author Frederico Silva <fredericosilva@ua.pt>
* @todo
* @deprecated Incomplete implementation, to be removed in Dicoogle 4
*/
@Deprecated
public class WadoServlet extends HttpServlet {

@Override
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
<name>dicoogle-all</name>

<properties>
<jetty.version>9.0.3.v20130506</jetty.version>
<jetty.version>9.4.52.v20230823</jetty.version>
<restlet.version>2.1.2</restlet.version>
<dcm4che.version>2.0.29</dcm4che.version>
<slf4j.version>1.7.36</slf4j.version>
Expand Down

0 comments on commit cf3735d

Please sign in to comment.