[mod_cluster-issues] [JBoss JIRA] (MODCLUSTER-285) get_path_param in mod_proxy_cluster doesn't expect '; ' as separator

[Jean-Frederic Clere (JIRA)]

    [ https://issues.jboss.org/browse/MODCLUSTER-285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12672029#comment-12672029 ] 

Jean-Frederic Clere commented on MODCLUSTER-285:
------------------------------------------------

Use the httpd-2.2.x code fixes the problem... (Need to do it).
                
> get_path_param in mod_proxy_cluster doesn't expect ';' as separator
> -------------------------------------------------------------------
>
>                 Key: MODCLUSTER-285
>                 URL: https://issues.jboss.org/browse/MODCLUSTER-285
>             Project: mod_cluster
>          Issue Type: Bug
>    Affects Versions: 1.1.3.Final, 1.2.0.Final
>            Reporter: Stefano Nichele
>            Assignee: Jean-Frederic Clere
>             Fix For: 1.2.1.Beta1
>
>
> Current version of get_path_param doesn't expect ';' as path parameters separator so this urls are not correctly handled:
> /test.jsp;jsessionid=123123.NODE01;name=vale
> See for instance the mod_proxy_balancer implementation (in httpd 2.2.21):
> /* Retrieve the parameter with the given name
>  * Something like 'JSESSIONID=12345...N'
>  */
> static char *get_path_param(apr_pool_t *pool, char *url,
>                             const char *name, int scolon_sep)
> {
>     char *path = NULL;
>     char *pathdelims = "?&";
>     if (scolon_sep) {
>         pathdelims = ";?&";
>     }
>     for (path = strstr(url, name); path; path = strstr(path + 1, name)) {
>         path += strlen(name);
>         if (*path == '=') {
>             /*
>              * Session path was found, get it's value
>              */
>             ++path;
>             if (strlen(path)) {
>                 char *q;
>                 path = apr_strtok(apr_pstrdup(pool, path), pathdelims, &q);
>                 return path;
>             }
>         }
>     }
>     return NULL;
> }
> Severity of this bug to me is not so high since using more than one path parameters is not so common.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira