From 1c74d8501c02fdefc1a337eb05196c2e12c5f34e Mon Sep 17 00:00:00 2001 From: TheDudeFromCI Date: Wed, 4 Aug 2021 19:52:10 -0700 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Neocky --- src/handlers/handle_config.py | 4 +++- src/handlers/handle_ftp.py | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/handlers/handle_config.py b/src/handlers/handle_config.py index 7cb1c0f..a77a357 100644 --- a/src/handlers/handle_config.py +++ b/src/handlers/handle_config.py @@ -72,8 +72,10 @@ def createConfig(): config['SFTP - Remote Server']['Server'] = '0.0.0.0' config['SFTP - Remote Server']['Username'] = 'user' config['SFTP - Remote Server']['Password'] = 'password' - config['SFTP - Remote Server'][';'] = 'If a different Port for SFTP needs to be used' + config['SFTP - Remote Server'][';'] = 'If a different Port for SFTP needs to be used (Default: 22)' config['SFTP - Remote Server']['SFTPPort'] = '22' + config['SFTP - Remote Server'][';_'] = 'If a different Port for FTP needs to be used (Default: 21)' + config['SFTP - Remote Server']['FTPPort'] = '21' config['SFTP - Remote Server'][';_'] = 'Change the path below if the plugin folder path is different on the SFTP/FTP server (Change only if you know what you are doing)' config['SFTP - Remote Server']['PluginFolderOnServer'] = '/plugins' config['SFTP - Remote Server'][';__'] = 'If you want to use FTP instead of SFTP change to (False) else use (True)' diff --git a/src/handlers/handle_ftp.py b/src/handlers/handle_ftp.py index eb03403..f1ab4cf 100644 --- a/src/handlers/handle_ftp.py +++ b/src/handlers/handle_ftp.py @@ -12,7 +12,7 @@ def createFTPConnection(): configValues = configurationValues() try: ftp = ftplib.FTP() - ftp.connect(configValues.sftp_server, configValues.sftp_port) + ftp.connect(configValues.sftp_server, configValues.ftp_port) ftp.login(configValues.sftp_user, configValues.sftp_password) return ftp except UnboundLocalError: