Mailing List Archive

#1770: Incomplete parsing of -a options
#1770: Incomplete parsing of -a options
----------------------+-----------------------------------
Reporter: Dridi | Owner:
Type: defect | Status: new
Priority: normal | Milestone:
Component: varnishd | Version: trunk
Severity: trivial | Keywords: listen proxy protocol
----------------------+-----------------------------------
This obviously fails:
{{{
$ bin/varnishd/varnishd -d -a 0:0,HTTP/2,PROXY -n $(mktemp -d)
Error: Unknown protocol 'HTTP/2'
}}}

But not the other way around:
{{{
$ bin/varnishd/varnishd -d -a 0:0,PROXY,HTTP/2 -n $(mktemp -d)
Platform:
Linux,4.0.8-200.fc21.x86_64,x86_64,-jnone,-smalloc,-smalloc,-hcritbit
200 287
-----------------------------
Varnish Cache CLI 1.0
-----------------------------
Linux,4.0.8-200.fc21.x86_64,x86_64,-jnone,-smalloc,-smalloc,-hcritbit
varnish-trunk revision 6bb6523

Type 'help' for command list.
Type 'quit' to close CLI session.
Type 'start' to launch worker process.
}}}

I suppose it might be possible once H2 support lands to combine H?+PROXY,
but right now I could check that you have [0,1] protocol in {{{MAC_Arg}}}.
This can get confusing if you try to combine H1+PROXY.

--
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1770>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator

_______________________________________________
varnish-bugs mailing list
varnish-bugs@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-bugs
Re: #1770: Incomplete parsing of -a options [ In reply to ]
#1770: Incomplete parsing of -a options
----------------------------------+----------------------------------------
Reporter: Dridi | Owner: Poul-Henning Kamp <phk@…>
Type: defect | Status: closed
Priority: normal | Milestone:
Component: varnishd | Version: trunk
Severity: trivial | Resolution: fixed
Keywords: listen proxy |
protocol |
----------------------------------+----------------------------------------
Changes (by Poul-Henning Kamp <phk@…>):

* owner: => Poul-Henning Kamp <phk@…>
* status: new => closed
* resolution: => fixed


Comment:

In [c654ff91a1287d36ac5c61599cb4ae3d6be22236]:
{{{
#!CommitTicketReference repository=""
revision="c654ff91a1287d36ac5c61599cb4ae3d6be22236"
Complain if there are sub-arguments to the -a protocol specs which don't
understand them.

Fixes: #1770
}}}

--
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1770#comment:1>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator

_______________________________________________
varnish-bugs mailing list
varnish-bugs@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-bugs