You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Which I think should not work if any of the string values templated there have double-quote characters in them.
For example:
log_proxy_wrapper -S '.%Y%m%d_%H%M%S"' -O /tmp/test.log -- cat <<< test
I'd expect this to work for any valid strftime template, which can include double quotes, but it does not with current release version, with the following error:
Fix can probably be using a different g_spawn_* call, which passes through individual arguments to underlying exec syscall, instead of parsing them from a shell-like quoted string, which will always be unreliable and depending on that parsing logic.
The text was updated successfully, but these errors were encountered:
Currently log_proxy_wrapper creates log_proxy subprocess like this:
Which I think should not work if any of the string values templated there have double-quote characters in them.
For example:
I'd expect this to work for any valid strftime template, which can include double quotes, but it does not with current release version, with the following error:
Fix can probably be using a different g_spawn_* call, which passes through individual arguments to underlying exec syscall, instead of parsing them from a shell-like quoted string, which will always be unreliable and depending on that parsing logic.
The text was updated successfully, but these errors were encountered: