Skip to content

Azure server#133

Open
calvinp0 wants to merge 10 commits intomainfrom
AzureServer
Open

Azure server#133
calvinp0 wants to merge 10 commits intomainfrom
AzureServer

Conversation

@calvinp0
Copy link
Copy Markdown
Member

No description provided.

Comment thread t3/utils/ssh.py
- A high-level representation of a session with an SSH server.
"""
ssh = paramiko.SSHClient()
ssh.set_missing_host_key_policy(paramiko.AutoAddPolicy())

Check failure

Code scanning / CodeQL

Accepting unknown SSH host keys when using Paramiko

Setting missing host key policy to AutoAddPolicy may be unsafe.
Comment thread t3/runners/rmg_adapter.py Fixed
Comment thread t3/runners/rmg_adapter.py
if up_file['file_name'] == 'check.chk':
try:
shutil.copyfile(src=up_file['local'], dst=os.path.join(self.local_path, 'check.chk'))
except shutil.SameFileError:

Check notice

Code scanning / CodeQL

Empty except

'except' clause does nothing but pass and there is no explanatory comment.
Comment thread t3/utils/ssh.py
timestamp = None
try:
timestamp = self._sftp.stat(remote_file_path_1).st_mtime
except IOError:

Check notice

Code scanning / CodeQL

Empty except

'except' clause does nothing but pass and there is no explanatory comment.
Comment thread t3/utils/ssh.py Fixed
Comment thread t3/utils/ssh.py Fixed
Comment thread t3/utils/ssh.py Fixed
Comment thread t3/runners/rmg_adapter.py Fixed
Comment thread t3/runners/rmg_adapter.py Fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants