mlibbey commented on PR #13208:
URL: https://github.com/apache/trafficserver/pull/13208#issuecomment-5444596000
Agree with zwoop. I think the bug here is that when u4wrh runs on
```
cond %{REMAP_PSEUDO_HOOK} [AND]
cond %{CLIENT-HEADER:X-Thisdoesntcache} ="" [NOT]
set-config proxy.config.http.cache.http INT 0
```
it yields the invalid
```
REMAP {
if inbound.req.X-Thisdoesntcache {
set-config("proxy.config.http.cache.http", "INT", 0);
}
}
```
if you do the double conversion, hrw4u correctly errors:
```
<stdin>:3:8: error: Invalid number of arguments (expected 2, got 3)
3 | set-config("proxy.config.http.cache.http", "INT", 0);
| ^
```
Also interesting adding the [L] after set-config and giving to u4wrh...
```
cond %{REMAP_PSEUDO_HOOK} [AND]
cond %{CLIENT-HEADER:X-Thisdoesntcache} ="" [NOT]
set-config proxy.config.http.cache.http 0 [L]
```
does not yield a `break;`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
