berekuk / questhub

Questhub.io source code
http://questhub.io
Other
79 stars 26 forks source link

Feedback on suspicious comments in Chef scripts #156

Open akondasif opened 5 years ago

akondasif commented 5 years ago

Greetings,

I am a security researcher, who is looking for security smells in Chef scripts. I found instances where certain keywords such as TODO, HACK, FIXME, bug repository IDs, in comments within Chef scripts. According to the Common Weakness Enumeration organization this is a security weakness (CWE-546: Suspicious Comment https://cwe.mitre.org/data/definitions/546.html).

I am trying to find out if you agree with the findings. I think it is possible to have a nuanced perspective. Any feedback is appreciated.

Source:

  1. https://github.com/berekuk/questhub/blob/master/cookbooks/apt/providers/repository.rb