Attachment 16701 Details for Bug 34844 – Propose URL decoding of stickysession route

[patch] Propose URL decoding of stickysession route

urldecoderoute.patch (text/plain), 1.28 KB, created by Terje Sten Bjerkseth on 2005-10-15 19:11:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Terje Sten Bjerkseth

Created: 2005-10-15 19:11:21 UTC

Size: 1.28 KB

patch

obsolete

>Index: mod_proxy_balancer.c
>===================================================================
>--- mod_proxy_balancer.c	(revision 321347)
>+++ mod_proxy_balancer.c	(working copy)
>@@ -182,6 +182,28 @@
>     return NULL;
> }
> 
>+/*
>+ * Decodes a '%' escaped string, and returns the number of characters
>+ */
>+static int decodeenc(char *x)
>+{
>+  int i, j, ch;
>+
>+  if (x[0] == '\0')
>+    return 0;               /* special case for no characters */
>+  for (i = 0, j = 0; x[i] != '\0'; i++, j++) {
>+    /* decode it if not already done */
>+    ch = x[i];
>+    if (ch == '%' && apr_isxdigit(x[i + 1]) && apr_isxdigit(x[i + 2])) {
>+      ch = ap_proxy_hex2c(&x[i + 1]);
>+      i += 2;
>+    }
>+    x[j] = ch;
>+  }
>+  x[j] = '\0';
>+  return j;
>+}
>+
> static proxy_worker *find_session_route(proxy_balancer *balancer,
>                                         request_rec *r,
>                                         char **route,
>@@ -195,6 +217,7 @@
>     *route = get_path_param(r->pool, *url, balancer->sticky);
>     if (!*route)
>         *route = get_cookie_param(r, balancer->sticky);
>+    (void)decodeenc(*route);
>     ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, r->server,
>                             "proxy: BALANCER: Found value %s for "
>                             "stickysession %s", *route, balancer->sticky);

Actions: View | Diff

Attachments on bug 34844: 16701 | 16702