[PATCH] Get rid of ban_url/ban.url

[Tollef Fog Heen]

]] Poul-Henning Kamp

> --------
> In message <1353665181-23260-1-git-send-email-tfheen at varnish-software.com>, Tol
> lef Fog Heen writes:
> 
> >ban.url is confusing as it takes a regular expression rather than a
> >fixed string, so get rid of it in favour of people being explicit and
> >using ban req.url ~ /foo
> 
> My only reservation is that backwards-compat issue.

Yeah, on the other hand, I see people getting confused by it, and the
change when upgrading is pure syntax.

> When did we deprecate ban.url ?  Was that in 2.x ?  If so, go for it...

It hasn't really been marked as deprecated (so we could still allow it,
just remove the docs for it) and then remove the implementation in the
next version.  ban() (or rather, purge()) got added to some 2.x version,
at which point purge.url/ban.url was slightly deprecated.

-- 
Tollef Fog Heen
Technical lead | Varnish Software AS
t: +47 21 98 92 64
We Make Websites Fly!