incubator-chemistry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dave Landers (JIRA)" <>
Subject [jira] Commented: (CMIS-210) test browser links need url encoding
Date Thu, 20 May 2010 16:57:20 GMT


Dave Landers commented on CMIS-210:

When the servlet retrieves the url query parameter with       
     String browseUrl = req.getParameter(PARAM_URL);
it is unencoded by the HttpServletRequest implementation.

So for symmetry, it seems that the entire query param should be encoded when the href's are
generated (by the xsl).

Doing this encoding and allowing the request to decode, the browseUrl will receive the exact
same thing as the repository provided in its link.

It is possible that then the issue is the section in the servlet which is commented "re-encode
parameters"?  This seems like it is doing the extra encoding - and too late, too (link parameters
have already been lost unless the entire url param is encoded)?

> test browser links need url encoding
> ------------------------------------
>                 Key: CMIS-210
>                 URL:
>             Project: Chemistry
>          Issue Type: Bug
>          Components: opencmis-test
>            Reporter: Dave Landers
>            Priority: Minor
>         Attachments:
> Links in the browser are not working with our CMIS repo (Oracle) as the url query param
is not encoded (so query params on our links - especially paging links next/previous/first/last
- are being lost)
> All the link and collection href's in the browser's xsl need to be encoded so they forward
to the repository properly.
> I have a potential fix for entry.xsl, feed.xsl, and service.xsl I would be happy to contribute

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message