@elextr okay it might be hard-ish to quote properly, but it's impossible for the user to escape properly. Just plain impossible. If the %f
expanded to i.e. foo"bar'baz
or worse, 'foo $(rm -rf ~ 2>/dev/null) bar'
(or without the quotes that are meant to create the injection in case it's surrounded by '
already).
You can s/quote/escape/
in my comment if you prefer, but that's the same deal.
And yes, we could just not care and hope it's all fine. Not sure if it's very sensible though.
—
Reply to this email directly or view it on GitHub.