Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Existing servlets cannot be edited to point to a different JSP pages in v7.0.0 #20570

Closed
sanjulamadurapperuma opened this issue Jun 21, 2024 · 1 comment
Assignees
Labels
7.0.0-migration Affected/7.0.0 Type/Bug ui Issues reported in the user interface of the embedded portals (Ex: Myaccount, Console)
Milestone

Comments

@sanjulamadurapperuma
Copy link
Contributor

Describe the issue:

Currently there is a way to configure custom servlets via the following deployment.toml configuration.

[[accountrecoveryendpoint.servlet]]
name = "<name>"
url = "<url>"
jsp = "<jsp>"

However, if the above configuration is utilized to edit an existing servlet, the following error occurs.

ERROR {org.apache.tomcat.util.digester.Digester} - End event threw exception java.lang.reflect.InvocationTargetException
...
Caused by: java.lang.IllegalArgumentException: The servlets named [recoverpassword.do] and [recoverpassword.do] are both mapped to the url-pattern [/recoverpassword.do] which is not permitted
...
ERROR {org.apache.tomcat.util.descriptor.web.WebXmlParser} - Parse error in application web.xml file at [file:/WEB-INF/web.xml] org.xml.sax.SAXParseException; systemId: file:/WEB-INF/web.xml; lineNumber: 260; columnNumber: 23; Error at line [260] column [23]: [The servlets named [recoverpassword.do] and [recoverpassword.do] are both mapped to the url-pattern [/recoverpassword.do] which is not permitted]
...
Caused by: java.lang.IllegalArgumentException: The servlets named [recoverpassword.do] and [recoverpassword.do] are both mapped to the url-pattern [/recoverpassword.do] which is not permitted
...

This indicates that existing servlets cannot be added using the same configuration utilized to add custom ones. This needs to be fixed since any migrations that occur from older versions to newer versions that had previously edited existing servlets will have to edit the relevant .xml.j2 file in order to persist that change, which is not a feasible solution in the long term.

Optional Fields

Related issues:

Suggested labels:

@nilasini
Copy link
Contributor

@bhagyasakalanka are we good to close this issue?

@nilasini nilasini added this to the 7.1.0-m1 milestone Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
7.0.0-migration Affected/7.0.0 Type/Bug ui Issues reported in the user interface of the embedded portals (Ex: Myaccount, Console)
Projects
None yet
Development

No branches or pull requests

3 participants