Исправлена ошибка указания порта в командах sftp-сервера #175
Conversation
WalkthroughИзменения в модуле Changes
Sequence DiagramsequenceDiagram
participant Client
participant SFTPModule
participant RemoteServer
Client->>SFTPModule: Вызов функции с параметрами соединения
SFTPModule->>SFTPModule: Формирование командной строки с портом
SFTPModule->>RemoteServer: Выполнение SFTP-операции
RemoteServer-->>SFTPModule: Результат операции
SFTPModule-->>Client: Возврат результата
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
src/cf/CommonModules/пбп_ИнтеграцииFTPSFTP/Ext/Module.bsl (3)
1233-1235: Проверка корректности путей при загрузке файла
Логика аналогична предыдущим участкам кода. Убедитесь, что переменнаяПутьФайлаНаСерверене содержит пробелов или особых символов без экранирования.
1313-1313: Использование порта в команде получения
Всё выглядит корректно с точки зрения вставки формата порта; убедитесь, что при больших номерах портов (например, выше 9999) обработка также корректна.
1397-1397: Загрузка файла с использованием параметра порта
Подстановка порта и учёт авторизационных данных осуществляются корректно. Рекомендуется проверить обработку ошибок сети и таймаутов.
[approve]
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/cf/CommonModules/пбп_ИнтеграцииFTPSFTP/Ext/Module.bsl(11 hunks)
🔇 Additional comments (8)
src/cf/CommonModules/пбп_ИнтеграцииFTPSFTP/Ext/Module.bsl (8)
1207-1209: Проверка формируемого URL-адреса важна
Все параметры для формирования строки подключения передаются корректно, но рекомендуется убедиться, что в случае пробелов или специальных символов в путях будет использоваться экранирование.
1220-1222: Единообразное формирование строки
Параметры для получения содержимого каталога сформированы в том же стиле, что и в предыдущем вызове curl. С точки зрения логики всё выглядит согласованно.
1249-1249: Удаление файла через curl
Команда rm для SFTP корректна, причём порт теперь учитывается через параметр %2. Всё выглядит штатно.
1292-1292: Проверка экранирования при чтении каталога
Убедитесь, что имя каталога в ПараметрыСоединения.КаталогИсходящие не содержит символов, требующих экранирования в контексте sshpass.
1334-1334: Параметр порта при удалении файла
Добавление формирования порта через Формат(ПараметрыСоединения.Порт, "ЧГ=0") сделано по аналогии. Нет замечаний.
1357-1357: Включение пароля и порта в команду отправки
Скрипт на expect корректно передаёт порт через ключ -P. Убедитесь, что пароль хранится защищённо.
1377-1377: Обход каталога через команду expect
Логика чтения каталога с учётом номера порта выглядит правильно. Наблюдений по ошибкам нет.
1418-1418: Удаление файла по expect с указанием порта
Использование -P в команде sftp соответствует требованиям, всё остальное в порядке.
#174
Summary by CodeRabbit