myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From leve ┼čipal (JIRA) <>
Subject [jira] [Commented] (PORTLETBRIDGE-217) Test URL should be "resources/myImage.jpg?javax.portlet.faces.BackLink=myBackLinkParam"
Date Tue, 06 Dec 2016 12:20:59 GMT


leve ┼čipal commented on PORTLETBRIDGE-217:
------------------------------------------ - Sexy lingerie black red White Lace open crotch bra G string sleepwear women
underbust nightwear teddy lingerie set S-XXL Visit the image link 

> Test URL should be "resources/myImage.jpg?javax.portlet.faces.BackLink=myBackLinkParam"
> ---------------------------------------------------------------------------------------
>                 Key: PORTLETBRIDGE-217
>                 URL:
>             Project: MyFaces Portlet Bridge
>          Issue Type: TCK Challenge
>          Components: TCK
>    Affects Versions: 2.0.0
>            Reporter: Wesley Hales
>            Assignee: Michael Freedman
>         Attachments: pbr-tck.patch, pbr-tck.patch
> [Alexander Smirnov & Wesley Hales: Red Hat]
> [jsr329-1.0.0]
> [portlet_2_bridge-1_0-final-spec]
> [chapter_6/section_6_5_1/Tests#testImplicitObject,
> chapter_6/section_6_1_3_1/Tests#illegalRedirectRenderTest,
> chapter_4/section_4_2_1/InitMethodTestPortlet#init,
> /testsuite/beans/NonJSFViewBean#getUrl]
> [
> Many tests failed because of possible TCK issue, that threats URLs not started with application
context as internals.
> Following this logic (and given Oracle's use of encodeActionURL to encode its equivalent
of the OutputLink) then encodeAction should do a return portletResponse.encodeURL(target)
if the url begins with a / and isn't (this application) context-path prefixed. I.e. we need
the bridge code to give the portlet container a chance to encode the server/port (construct
and absolute url) - typically when the portlet container is remote.
> Sound about right? I know the spec doesn't say anything about this but it seems to fall
out from this decision/interpretation. Does your code already do this? If not should it?
> Mike
> On 6/14/2011 1:29 PM, Michael Freedman wrote:
> > So what you are saying is that because ViewHandler.getResourceURL() is defined and
it will always append the ContextPath then the spec/impl should assume that urls starting
with '/' that don't contain a ContextPath must be out of the application (because all in application
urls would have been channeled through getResourceURL? I.e. just using /xxx as a resource
url without a context path doesn't work in servlet based JSF. Interesting - okay I see your
logic. Please file a JIRA against the TCK indicating the test url it should use ought to be
"resources/myImage.jpg?javax.portlet.faces.BackLink=myBackLinkParam". If possible please make
the needed changes yourself and verify (and then submit the jira with either the details of
the code change or a patch file).
> >
> > I will file a bug against the RI concerning its not treating / as an external url
- and since we are changing the test/fixing a bug will just include in the 2.0.1 release.
> > Mike
> ]

This message was sent by Atlassian JIRA

View raw message