Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 269678

Summary: [Branding] Parameter Lost during Redirection
Product: [RT] RAP Reporter: Walter Mising name <kwchess>
Component: WorkbenchAssignee: Project Inbox <rap-inbox>
Status: RESOLVED WONTFIX QA Contact:
Severity: normal    
Priority: P3 CC: ivan
Version: 1.2   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
RAP Application Sample none

Description Walter Mising name CLA 2009-03-23 07:53:08 EDT 
Created attachment 129584 [details]
RAP Application Sample

Here it is.

Steps to reproduce:
-------------------
Launch the attached RAP application
Open the browser and type http://localhost/ext?l=1234
The output will be fine similar to
SET Show_Property Value: cguutmtc2380 1234
SessionID = cguutmtc2380 LP = 1234

Failure Scenario ( Branding )
------------------------------
Edit ExternalHitServlet.java
Change
    response.sendRedirect("/rap?startup=test") ;
//    response.sendRedirect("/show.do") ;
to
//    response.sendRedirect("/rap?startup=test") ;
    response.sendRedirect("/show.do") ;

Launch the RAP application
Try: http://localhost/ext?l=123456
LP is null as seen in the Console:

SET Show_Property Value: 1phr025sjjszw 123456
SessionID = 1phr025sjjszw LP = null

Thanks,
Best Regards,
Walter G. Prabhakar.
Comment 1 Ivan Furnadjiev CLA 2009-03-31 05:19:09 EDT 
Walter, to pass the parameters on redirect just put them in the URL:
response.sendRedirect("/show.do?l="+s) ;
instead of putting them in the session.
Comment 2 Walter Mising name CLA 2009-03-31 08:05:21 EDT 
Actually, I don't want the user to see /show.do?l=<xxxxx> in the URL.
Definitely there is some difference in the way it done with /rap?startup=test and /show.do. 
While the workaround is good enough to address the problem, it still is a bug.
It would only be appropriate if the resolution is set to "WONTFIX" instead of "INVALID".
Comment 3 Rüdiger Herrmann CLA 2009-04-01 04:36:58 EDT 
We can't do much about this without reworking a lot of code (if ever).
Given that there is a reasonable workaround I will mark the bug as WONTFIX.