Bug 1027372 - Updated jsf-impl 1.2 in ER7 breaks backwards compatibility, Seam 2.2 examples won't run
Summary: Updated jsf-impl 1.2 in ER7 breaks backwards compatibility, Seam 2.2 examples...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: JSF
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: CR1
: EAP 6.2.0
Assignee: David Walluck
QA Contact: Marek Schmidt
Russell Dickenson
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-06 16:48 UTC by Ron Šmeral
Modified: 2016-11-01 01:37 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-12-15 16:19:29 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Reproducer (7.77 MB, application/zip)
2013-11-06 16:48 UTC, Ron Šmeral
no flags Details
server.log (3.07 MB, text/x-log)
2013-11-06 16:51 UTC, Ron Šmeral
no flags Details

Description Ron Šmeral 2013-11-06 16:48:42 UTC
Created attachment 820567 [details]
Reproducer

Description of problem:
Seam 2.2 examples fail to run on EAP 6.2.0.ER7 (they do deploy), throwing StackOverflowErrors, FacesExceptions and ServletExceptions, all related to JSF. 

Version-Release number of selected component (if applicable):
6.2.0.ER7

Steps to Reproduce:
1. Deploy a Seam 2.2 example (migrated to EAP6 as per EAP documentation), e.g. the blog example (EAR attached)
2. Open http://localhost:8080/seam-blog/

Actual results:
HTTP 500 error, javax.servlet.ServletException: java.lang.IllegalArgumentException: no file extension in servlet path: /

Expected results:
The index page of the blog example is loaded.

Additional info: 
The problem seems to be related to the updated JSF implementation (jsf-impl-1.2_15-b01-redhat-4 in ER6, jsf-impl-1.2_15-b01-redhat-6 in ER7). If the -redhat-4 version is used in ER7, the problem doesn't appear.

Comment 1 Ron Šmeral 2013-11-06 16:50:14 UTC
It's actually jsf-impl-1.2_15-b01-redhat-3 in ER6.

Comment 2 Ron Šmeral 2013-11-06 16:51:15 UTC
Created attachment 820568 [details]
server.log

Comment 3 Stan Silvert 2013-11-06 17:08:17 UTC
I'm reassigning this one to Farah.

The only reason JSF 1.2 exists in EAP6 is to support old Seam 2 apps.  So I don't have a problem with going back to an old release if that's what makes Seam 2 work.  

However, I can't recall why the version was updated in the first place.  So somebody needs to do some research to find out why the JSF 1.2 version was upgraded.  My only hesitation with downgrading is that we might break whatever it was that prompted the upgrade to jsf-impl-1.2_15-b01-redhat-6.

The other strange thing is that we should be talking about the same code base.  All these versions we are talking about are based on Mojarra 1.2_15-b01.  So what is the difference between readhat-3, redhat-4, redhat-5, and redhat-6? 

Farah, please take point on this.  I'll be available to answer questions.

Comment 4 Farah Juma 2013-11-06 20:04:49 UTC
I tracked down the PR that upgraded from jsf-impl-1.2_15-b01-redhat-3 to jsf-impl-1.2_15-b01-redhat-6: 
https://github.com/jbossas/jboss-eap/pull/618

The PR references Bug #957687 (an issue with differences in the contents of jsf-impl.jar when comparing an RPM install to a ZIP install). This issue was marked as a blocker for EAP 6.2.0. 

David, please provide some more information about the upgrade to jsf-impl-1.2_15-b01-redhat-6. What is the difference between redhat-3 and redhat-6?

Comment 5 Marek Schmidt 2013-11-07 16:31:19 UTC
Similar errors can be reproduced with a Spring application using RichFaces 3.3, so it is not Seam-specific.

Comment 6 Farah Juma 2013-11-07 16:59:05 UTC
There was an issue with the jsf-impl-1.2_15-b01-redhat-6 JAR itself. I talked to David about this yesterday and he mentioned that they recently switched to using a Maven build and a patch didn't get applied cleanly. He created a new redhat-7 version and the Seam example now works for me with the JARs from this build:

https://brewweb.devel.redhat.com/taskinfo?taskID=6544144

Comment 7 Farah Juma 2013-11-07 18:16:02 UTC
Reassigning this one to David to try to get the updated build included in EAP 6.2.0 CR1.

Comment 8 David Walluck 2013-11-07 18:49:09 UTC
Created pull request: <https://github.com/jbossas/jboss-eap/pull/677>.

Comment 9 David Walluck 2013-11-07 18:56:56 UTC
This was a conflict between <https://issues.jboss.org/browse/WFLY-685> and <https://issues.jboss.org/browse/JBPAPP-6414> which are fixes for the same issue, but apparently WFLY-685 does not work for EAP.

Comment 10 Ron Šmeral 2013-11-08 14:07:55 UTC
I can confirm that the redhat-8 version fixes the issue for Seam.

Comment 11 Ron Šmeral 2013-11-11 15:22:56 UTC
This issue was verified using the 6.2.0.CR1 preview bits.


Note You need to log in before you can comment on or make changes to this bug.