[
https://issues.apache.org/jira/browse/WW-5141?focusedWorklogId=890355&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-890355
]
ASF GitHub Bot logged work on WW-5141:
--------------------------------------
Author: ASF GitHub Bot
Created on: 13/Nov/23 21:40
Start Date: 13/Nov/23 21:40
Worklog Time Spent: 10m
Work Description: sepe81 commented on code in PR #778:
URL: https://github.com/apache/struts/pull/778#discussion_r1391731445
##########
plugins/velocity/pom.xml:
##########
@@ -54,7 +54,7 @@
<artifactId>mockobjects-core</artifactId>
<scope>test</scope>
</dependency>
-
+
Review Comment:
```suggestion
```
revert change
##########
apps/showcase/pom.xml:
##########
@@ -99,8 +94,8 @@
</dependency>
<dependency>
- <groupId>javax.servlet</groupId>
- <artifactId>javax.servlet-api</artifactId>
+ <groupId>jakarta.servlet</groupId>
+ <artifactId>jakarta.servlet-api</artifactId>
Review Comment:
```suggestion
<artifactId>jakarta.servlet-api</artifactId>
```
##########
pom.xml:
##########
@@ -792,50 +748,56 @@
</dependency>
<dependency>
- <groupId>javax.servlet</groupId>
- <artifactId>javax.servlet-api</artifactId>
- <version>3.1.0</version>
+ <groupId>jakarta.servlet</groupId>
+ <artifactId>jakarta.servlet-api</artifactId>
+ <version>6.0.0</version>
<scope>provided</scope>
</dependency>
<dependency>
- <groupId>org.glassfish</groupId>
- <artifactId>javax.el</artifactId>
- <version>3.0.1-b12</version>
+ <groupId>org.glassfish</groupId>
+ <artifactId>jakarta.el</artifactId>
+ <version>5.0.0-M1</version>
</dependency>
<dependency>
- <groupId>taglibs</groupId>
- <artifactId>standard</artifactId>
- <version>1.1.2</version>
+ <groupId>org.glassfish.web</groupId>
+ <artifactId>jakarta.servlet.jsp.jstl</artifactId>
+ <version>3.0.1</version>
<scope>test</scope>
</dependency>
<dependency>
- <groupId>javax.servlet</groupId>
- <artifactId>jstl</artifactId>
- <version>1.2</version>
+ <groupId>jakarta.servlet.jsp.jstl</groupId>
+
<artifactId>jakarta.servlet.jsp.jstl-api</artifactId>
+ <version>3.0.0</version>
<scope>test</scope>
</dependency>
+
+ <dependency>
+ <groupId>jakarta.el</groupId>
+ <artifactId>jakarta.el-api</artifactId>
+ <version>5.0.0</version>
+ </dependency>
<dependency>
<groupId>org.apache.tomcat</groupId>
<artifactId>tomcat-jasper</artifactId>
- <version>8.5.53</version>
+ <version>10.1.15</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.apache.tomcat</groupId>
<artifactId>tomcat-api</artifactId>
- <version>8.5.53</version>
+ <version>10.1.15</version>
<scope>provided</scope>
</dependency>
<dependency>
- <groupId>javax.servlet.jsp</groupId>
- <artifactId>jsp-api</artifactId>
- <version>2.1</version>
+ <groupId>jakarta.servlet.jsp</groupId>
+
<artifactId>jakarta.servlet.jsp-api</artifactId>
+ <version>3.1.0</version>
Review Comment:
```suggestion
<version>3.1.0</version>
```
##########
pom.xml:
##########
@@ -946,38 +908,12 @@
<artifactId>spring-web</artifactId>
<version>${spring.platformVersion}</version>
</dependency>
-
- <dependency>
+ <dependency>
Review Comment:
revert line
##########
pom.xml:
##########
@@ -792,50 +748,56 @@
</dependency>
<dependency>
- <groupId>javax.servlet</groupId>
- <artifactId>javax.servlet-api</artifactId>
- <version>3.1.0</version>
+ <groupId>jakarta.servlet</groupId>
+ <artifactId>jakarta.servlet-api</artifactId>
+ <version>6.0.0</version>
<scope>provided</scope>
</dependency>
<dependency>
- <groupId>org.glassfish</groupId>
- <artifactId>javax.el</artifactId>
- <version>3.0.1-b12</version>
+ <groupId>org.glassfish</groupId>
+ <artifactId>jakarta.el</artifactId>
+ <version>5.0.0-M1</version>
</dependency>
<dependency>
- <groupId>taglibs</groupId>
- <artifactId>standard</artifactId>
- <version>1.1.2</version>
+ <groupId>org.glassfish.web</groupId>
+ <artifactId>jakarta.servlet.jsp.jstl</artifactId>
+ <version>3.0.1</version>
<scope>test</scope>
</dependency>
<dependency>
- <groupId>javax.servlet</groupId>
- <artifactId>jstl</artifactId>
- <version>1.2</version>
+ <groupId>jakarta.servlet.jsp.jstl</groupId>
+
<artifactId>jakarta.servlet.jsp.jstl-api</artifactId>
+ <version>3.0.0</version>
<scope>test</scope>
</dependency>
+
+ <dependency>
+ <groupId>jakarta.el</groupId>
+ <artifactId>jakarta.el-api</artifactId>
+ <version>5.0.0</version>
+ </dependency>
<dependency>
<groupId>org.apache.tomcat</groupId>
<artifactId>tomcat-jasper</artifactId>
- <version>8.5.53</version>
+ <version>10.1.15</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.apache.tomcat</groupId>
<artifactId>tomcat-api</artifactId>
- <version>8.5.53</version>
+ <version>10.1.15</version>
<scope>provided</scope>
</dependency>
<dependency>
- <groupId>javax.servlet.jsp</groupId>
- <artifactId>jsp-api</artifactId>
- <version>2.1</version>
+ <groupId>jakarta.servlet.jsp</groupId>
+
<artifactId>jakarta.servlet.jsp-api</artifactId>
Review Comment:
```suggestion
<artifactId>jakarta.servlet.jsp-api</artifactId>
```
tabs2spaces
##########
bundles/pom.xml:
##########
@@ -51,9 +51,9 @@
</dependency>
- <dependency>
- <groupId>javax.servlet</groupId>
- <artifactId>javax.servlet-api</artifactId>
+ <dependency>
Review Comment:
```suggestion
<dependency>
```
trim trailing space
Issue Time Tracking
-------------------
Worklog Id: (was: 890355)
Time Spent: 1h 50m (was: 1h 40m)
> Support for JEE 9+
> -------------------
>
> Key: WW-5141
> URL: https://issues.apache.org/jira/browse/WW-5141
> Project: Struts 2
> Issue Type: New Feature
> Components: Core
> Reporter: Daniel Le Berre
> Priority: Major
> Fix For: 7.0.0
>
> Attachments: pom.xml
>
> Time Spent: 1h 50m
> Remaining Estimate: 0h
>
> JEE 9 breaks the JEE API by replacing javax domain by jakarta.
> Tomcat 10 implements some specifications of JEE 9.
> Struts 2.5 has some dependencies with the javax servlet API.
> Struts would require some changes to run on Tomcat 10+.
> Is there any plan to support JEE 9+ in the future?
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
