<div dir="ltr">One of the things Dag pointed about this is that if you want to hash based on the client IP address you will need to do "" + client.ip. <br>This is a valid concern although the only situation I can think of where client.identity will be missed.<br><br>There are a few options:<br><br>1. Use "" + client.ip. Somewhat ugly but will do the job.<br><br>2. Do automatic conversion for arguments when the expected type is STRING or STRING_LIST. See diff below.<br>We do this already in a number of places but not for arguments.<br><br>3. Stash it in the req.http space. e.g. <br><br>set req.http.session = client.ip;<br>set req.backend_hint = hash.backend(req.http.session);<br>#unset req.http.session;<br><br>4. Change the hash director to use client.identity again. e.g. drop the STRING_LIST and use client.identity.<br><br>5. Do nothing.<br><br>As things are in master, client.identity lost it's place and it's more like a scratch space or throw away variable and feels like a special case but YMMV.<br><br>Any opinions?<br><br>f.-<br><br>diff --git a/lib/libvcc/vcc_expr.c b/lib/libvcc/vcc_expr.c                      <br>index b9c6531..4019fec 100644<br>--- a/lib/libvcc/vcc_expr.c<br>+++ b/lib/libvcc/vcc_expr.c<br>@@ -609,6 +609,9 @@ vcc_do_arg(struct vcc *tl, struct func_arg *fa)<br>        } else {<br>                vcc_expr0(tl, &e2, fa->type);<br>                ERRCHK(tl);<br>+               if (e2->fmt != fa->type &&<br>+                   (fa->type == STRING || fa->type == STRING_LIST))<br>+                       vcc_expr_tostring(tl, &e2, fa->type);<br>                if (e2->fmt != fa->type) {<br>                        VSB_printf(tl->sb, "Wrong argument type.");<br>                        VSB_printf(tl->sb, "  Expected %s.",<br><br><br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 2, 2015 at 1:48 PM, Martin Blix Grydeland <span dir="ltr"><<a href="mailto:martin@varnish-software.com" target="_blank">martin@varnish-software.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yeah, this is OBE now.<div><br></div><div>Martin </div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On 1 July 2015 at 15:58, Dridi Boukelmoune <span dir="ltr"><<a href="mailto:dridi@varni.sh" target="_blank">dridi@varni.sh</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div><div>On Wed, Jul 1, 2015 at 3:05 PM, Federico Schwindt <<a href="mailto:fgsch@lodoss.net" target="_blank">fgsch@lodoss.net</a>> wrote:<br>
> Hi,<br>
><br>
> The hash director needs a string now so this no longer needed.<br>
><br>
> Opinions?<br>
<br>
</div></div>Agreed<br>
<br></div></div>
_______________________________________________<br>
varnish-dev mailing list<br>
<a href="mailto:varnish-dev@varnish-cache.org" target="_blank">varnish-dev@varnish-cache.org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev" rel="noreferrer" target="_blank">https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev</a><span class="HOEnZb"><font color="#888888"><br>
</font></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><div><table style="font-size:12px;line-height:1.5em;font-family:'Helvetica Neue',Arial,sans-serif;color:rgb(102,102,102);width:550px;border-top-width:1px;border-top-style:solid;border-top-color:rgb(238,238,238);border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:rgb(238,238,238);margin-top:20px;padding-top:5px;padding-bottom:5px" border="0" cellpadding="0" cellspacing="0"><tbody><tr><td width="100"><a href="http://varnish-software.com" target="_blank"><img src="http://www.varnish-software.com/static/media/logo-email.png"></a><span></span><span></span></td><td><strong style="font-size:14px;color:rgb(34,34,34)">Martin Blix Grydeland</strong><br>Senior Developer | Varnish Software AS<br>Mobile: <a href="tel:%2B47%20992%2074%20756" value="+4799274756" target="_blank">+47 992 74 756</a><br><span style="font-weight:bold">We Make Websites Fly!</span></td></tr></tbody></table></div></div></div></div>
</font></span></div>
</blockquote></div><br></div>