Merck / metalite.ae

An R package for standard adverse events analysis
http://merck.github.io/metalite.ae/
GNU General Public License v3.0
17 stars 4 forks source link

Design Pattern: using `rlang::eval_tidy()` in a secure way. #153

Closed fb-elong closed 1 year ago

fb-elong commented 1 year ago

Is rlang::eval_tidy() evaluating arbitrary input expressions? That would create a security vulnerbility, as the expression can be malicious.

nanxstats commented 1 year ago

I did some preliminary research and wrote a draft.

The basics seem clear but I wonder if there is a more authoritative guide on this topic.

nanxstats commented 1 year ago

We can close this for now. We can analyze the use cases on case by case basis. It's a bit challenging to write a comprehensive secure evaluation guide from a non-rlang author's perspective.