__key_value fails if initial value contains metacharacters #321

New Issue

ungleich-gitea · 2021-11-20T15:24:21Z

ungleich-gitea commented

2021-11-20 15:24:21 +00:00

Created by: phrawzty

The awk command which performs the essential function of __key_value will fail if the initial value contains metacharacters. For example :

# manifest
__key_value logtarget --value '/var/log/foo.log' --file 'file.cfg' --delimiter '='

# file.cfg
logtarget=/var/log/bar.log

This will end up generating an awk gsub string that contains the following :

[...] {gsub(//var/log/thing.log/, "/var/log/foo.log")} [...]

The first element is clearly illegal and will result in a syntax failure from awk.

A solution is to replace the slashes which surround the first element at line 43 of /conf/type/__key_value/gencode-remote with double-quotes, as is already the case with the second element. Is there any particular reason why slashes are preferred here over double-quotes ?

*Created by: phrawzty* The `awk` command which performs the essential function of `__key_value` will fail if the _initial_ value contains metacharacters. For example : ``` # manifest __key_value logtarget --value '/var/log/foo.log' --file 'file.cfg' --delimiter '=' ``` ``` # file.cfg logtarget=/var/log/bar.log ``` This will end up generating an awk gsub string that contains the following : ``` [...] {gsub(//var/log/thing.log/, "/var/log/foo.log")} [...] ``` The first element is clearly illegal and will result in a syntax failure from `awk`. A solution is to replace the slashes which surround the first element at line 43 of `/conf/type/__key_value/gencode-remote` with double-quotes, as is already the case with the second element. Is there any particular reason why slashes are preferred here over double-quotes ?

ungleich-gitea closed this issue

2021-11-20 15:24:21 +00:00

ungleich-gitea commented

2021-11-20 15:53:24 +00:00

Created by: phrawzty

Hehe, yes, that is exactly the solution. :)

*Created by: phrawzty* Hehe, yes, that is exactly the solution. :)

ungleich-gitea commented

2021-11-20 15:53:26 +00:00

Created by: telmich

Should be fixed in 62b5af8, can you verify that it works for you?

*Created by: telmich* Should be fixed in 62b5af8, can you verify that it works for you?