VTS doesn't support query string encoding
vultj opened this issue · 1 comments
vultj commented
If query strings are "encoded", *
is turned into %2A
. This is not supported by vts, see below:
# curl '127.0.0.1:9001/status/control?cmd=reset&group=%2A'
{"processingReturn":false,"processingCommandString":"reset","processingGroupString":"%2A","processingZoneString":"","processingCounts":0}
# curl '127.0.0.1:9001/status/control?cmd=reset&group=*'
{"processingReturn":true,"processingCommandString":"reset","processingGroupString":"*","processingZoneString":"","processingCounts":8}
u5surf commented
@vultj
Thanks reporting.
Indeed, although it seems to be a some kind of specification because the other characters (e.g. @) does not support escaping and it should restrict to display the data properly.
We'll consider whether it has a validity to add the rule of escaped character by confirming the other usecase or the another nginx's modules.
nginx-module-vts/src/ngx_http_vhost_traffic_status_display.c
Lines 126 to 160 in bd33883
Moreover, could you tell me what specifically do you have trouble with?
I wanna know by what had been escaped the query string.