Nagios-Plugins
Nagios-Plugins copied to clipboard
Update check_sftp.py
Update for support Password and fix some bugs. Added option to find text in file.
Kudos, SonarCloud Quality Gate passed!
0 Bugs
0 Vulnerabilities
0 Security Hotspots
7 Code Smells
No Coverage information
0.0% Duplication
This pull request introduces 2 alerts when merging 9029c29c6b0f47883b7b20cf289c77246f6cb27a into b577b20b298bd1497b9684309a80c1973866ff3b - view on LGTM.com
new alerts:
- 2 for Unused import
What is the use case for using a password instead of SSH Key? Is this less secure?
Also, I generally don't change the date and author headers as that is the origin time of the code, which is an important reference point because it gives a clue as to the code's style based on that time frame, I would write this very differently today.
Tbh if I was still using this code and going to be checking for more things like you're going I'd probably just write a new version of this code with paramiko - there is even a long-standing todo about this at the top of the code.