Add a counterexample to 'do it in Ruby'
This commit is contained in:
parent
71fce0b2f8
commit
97c5d38097
1 changed files with 6 additions and 0 deletions
|
@ -22,6 +22,12 @@ FileUtils.mkdir_p "tmp/special/directory"
|
||||||
contents = `cat #{filename}`
|
contents = `cat #{filename}`
|
||||||
# Correct
|
# Correct
|
||||||
contents = File.read(filename)
|
contents = File.read(filename)
|
||||||
|
|
||||||
|
# Sometimes a shell command is just the best solution. The example below has no
|
||||||
|
# user input, and is hard to implement correctly in Ruby: delete all files and
|
||||||
|
# directories older than 120 minutes under /some/path, but not /some/path
|
||||||
|
# itself.
|
||||||
|
Gitlab::Popen.popen(%W(find /some/path -not -path /some/path -mmin +120 -delete))
|
||||||
```
|
```
|
||||||
|
|
||||||
This coding style could have prevented CVE-2013-4490.
|
This coding style could have prevented CVE-2013-4490.
|
||||||
|
|
Loading…
Reference in a new issue