eclipse-ee4j / glassfish

Eclipse GlassFish
https://eclipse-ee4j.github.io/glassfish/
386 stars 144 forks source link

Mapping url-patterns to jsp documents is currently broken #7278

Closed glassfishrobot closed 15 years ago

glassfishrobot commented 15 years ago

Consider the following fragment in WEB-INF/web.xml:


*.jsp true

When a request is made to the JSP document, the following error is returned:

JasperException: /index.jsp(40,5) PWC6282: must not be used in standard syntax

This works find in V2, and was working (best I can tell) last week in the v3 trunk.

Environment

Operating System: All Platform: Macintosh

Affected Versions

[9.0pe]

glassfishrobot commented 6 years ago
glassfishrobot commented 15 years ago

@glassfishrobot Commented @rlubke said: Created an attachment (id=2329) test case

glassfishrobot commented 15 years ago

@glassfishrobot Commented jluehe said: ...

glassfishrobot commented 15 years ago

@glassfishrobot Commented @rlubke said: Created an attachment (id=2332) Correct application

glassfishrobot commented 15 years ago

@glassfishrobot Commented @rlubke said: The following exception is seen in the log:

EVERE: java.lang.NoSuchMethodException: com.sun.enterprise.deployment.JspGroupDescriptor.setIsXml(boolean) Mar 9, 2009 2:08:09 PM SEVERE: at java.lang.Class.getMethod(Class.java:1605) Mar 9, 2009 2:08:09 PM SEVERE: at com.sun.enterprise.deployment.node.DeploymentDescriptorNode.setDescriptorInfo(DeploymentDescriptorNode.java:460) Mar 9, 2009 2:08:09 PM SEVERE: at com.sun.enterprise.deployment.node.DeploymentDescriptorNode.setElementValue(DeploymentDescriptorNode.java:385) Mar 9, 2009 2:08:09 PM

glassfishrobot commented 15 years ago

@glassfishrobot Commented kchung said: Appears to be a DOL problem. Reassigned to Shing Wai.

glassfishrobot commented 15 years ago

@glassfishrobot Commented @rlubke said: Created an attachment (id=2339) patch

glassfishrobot commented 15 years ago

@glassfishrobot Commented @rlubke said: I've attached a patch that resolves the issue in my workspace.

glassfishrobot commented 15 years ago

@glassfishrobot Commented kchung said: Unfortunately, the patch assumes that the default for is-xml is false, which is incorrect. For instance, the default for is-xml is true for a JSP page with a jspx subfix.

One question though: I thought it's OK to pass a boolean to a method expecting a Boolean argument, because autoboxing should take care of it, no?

glassfishrobot commented 15 years ago

@glassfishrobot Commented @shingwaichan said: v3/deployment/dol/src/main/java/com/sun/enterprise/deployment/node/web $ svn commit JspGroupNode.java Sending JspGroupNode.java Transmitting file data . Committed revision 26012.

glassfishrobot commented 15 years ago

@glassfishrobot Commented File: jsf-guessNumber.war Attached By: @rlubke

glassfishrobot commented 15 years ago

@glassfishrobot Commented File: jsf-guessNumber2.0.war Attached By: @rlubke

glassfishrobot commented 15 years ago

@glassfishrobot Commented File: patch.txt Attached By: @rlubke

glassfishrobot commented 7 years ago

@glassfishrobot Commented This issue was imported from java.net JIRA GLASSFISH-7278

glassfishrobot commented 15 years ago

@glassfishrobot Commented Reported by @rlubke

glassfishrobot commented 15 years ago

@glassfishrobot Commented Marked as fixed on Wednesday, April 8th 2009, 9:25:44 am