Skip to content
This repository has been archived by the owner on Oct 6, 2020. It is now read-only.

Warn about shell=True in subprocess.run #24

Closed
xen0l opened this issue Sep 18, 2019 · 0 comments · Fixed by #25
Closed

Warn about shell=True in subprocess.run #24

xen0l opened this issue Sep 18, 2019 · 0 comments · Fixed by #25

Comments

@xen0l
Copy link
Contributor

xen0l commented Sep 18, 2019

subprocess.run was added in Python 3.5+ as a simple wrapper around subprocess.Popen function. It provides higher level API and official Python documnetation recommends to use it.

dlint should warn about it:

import subprocess
...
subprocess.run(cmd, shell=True)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant