Исправление ошибки форматирования полей сообщений sftp #186
Conversation
|
Warning Rate limit exceeded@thvvmas has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 18 minutes and 5 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
WalkthroughВ данном pull request выполнены изменения в модуле интеграционной службы. Основная модификация касается процедуры Changes
Assessment against linked issues
Possibly related PRs
Suggested reviewers
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
🔭 Outside diff range comments (1)
src/cf/CommonModules/пбп_ИнтеграцииСлужебный/Ext/Module.bsl (1)
Line range hint
299-312: Улучшить определение XML форматаИспользование
СтрНайти(сообщение, "xml")для определения XML формата может привести к ложным срабатываниям. Рекомендуется использовать более надежный метод проверки.Предлагаемое решение:
-Если Не ПустаяСтрока(СтруктураЗаписиИстории.ИсходящееСообщение) - И СтрНайти(СтруктураЗаписиИстории.ИсходящееСообщение, "xml") <> 0 Тогда +Если Не ПустаяСтрока(СтруктураЗаписиИстории.ИсходящееСообщение) + И ЭтоКорректныйXML(СтруктураЗаписиИстории.ИсходящееСообщение) ТогдаДобавьте новую функцию:
Функция ЭтоКорректныйXML(Текст) Попытка ЧтениеXML = Новый ЧтениеXML; ЧтениеXML.УстановитьСтроку(Текст); Возврат Истина; Исключение Возврат Ложь; КонецПопытки; КонецФункции
🧹 Nitpick comments (1)
src/cf/CommonModules/пбп_ИнтеграцииСлужебный/Ext/Module.bsl (1)
Line range hint
299-312: Устранить дублирование кодаКод обработки входящих и исходящих сообщений идентичен. Рекомендуется выделить общую логику в отдельную функцию.
Предлагаемое решение:
+Процедура ОбработатьСообщениеXML(Знач Сообщение) + Если Не ПустаяСтрока(Сообщение) + И СтрНайти(Сообщение, "xml") <> 0 Тогда + ОчиститьДлинныеАтрибутыСообщенияXML(Сообщение); + КонецЕсли; +КонецПроцедуры Процедура ДополнительноОбработатьЗапросыИнтеграцииПередЗаписьюВИсторию(СтруктураЗаписиИстории) // ... существующий код ... - Если Не ПустаяСтрока(СтруктураЗаписиИстории.ИсходящееСообщение) - И СтрНайти(СтруктураЗаписиИстории.ИсходящееСообщение, "xml") <> 0 Тогда - ОчиститьДлинныеАтрибутыСообщенияXML(СтруктураЗаписиИстории.ИсходящееСообщение); - КонецЕсли; - - Если Не ПустаяСтрока(СтруктураЗаписиИстории.ВходящееСообщение) - И СтрНайти(СтруктураЗаписиИстории.ВходящееСообщение, "xml") <> 0 Тогда - ОчиститьДлинныеАтрибутыСообщенияXML(СтруктураЗаписиИстории.ВходящееСообщение); - КонецЕсли; + ОбработатьСообщениеXML(СтруктураЗаписиИстории.ИсходящееСообщение); + ОбработатьСообщениеXML(СтруктураЗаписиИстории.ВходящееСообщение);
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/cf/CommonModules/пбп_ИнтеграцииСлужебный/Ext/Module.bsl(1 hunks)
🔇 Additional comments (1)
src/cf/CommonModules/пбп_ИнтеграцииСлужебный/Ext/Module.bsl (1)
291-297: Оптимизация обработки сообщений!Хорошее решение по добавлению ранней проверки типа интеграции. Это предотвращает ненужную обработку XML для форматов SFTP и командной строки, что улучшает производительность.
This comment has been minimized.
This comment has been minimized.
| Возврат; | ||
| КонецЕсли; | ||
|
|
||
| Если Не ПустаяСтрока(СтруктураЗаписиИстории.ИсходящееСообщение) |
There was a problem hiding this comment.
Слушай может проще просто проверять на формат интеграции? У нас же там есть XML, типа если ФорматИнтеграции = Перечисления.пбп_ФорматыИнтеграций.XML тогда обрабатывать иначе нет. Так универсальнее и не зависит от типа интеграции и не надо делать запрос на получение типа
There was a problem hiding this comment.
Формат интеграции заполняется программистом и если его заполнить не корректно - это не будет работать. Тут можно запутаться в том, какой формат указывать в некоторых типах интеграции. Например, на команду терминала мы получаем XML файл, который потом кладем во входящий запрос. Какой формат интеграции должен быть? SH/BAT как исходящее сообщение или XML как входящее сообщение?
There was a problem hiding this comment.
Ну это вопрос к тому что если программист косяк и назначил неправильно то просто не будет отражаться форматирование xml - сам виноват. Я думаю что лучше всё-таки привязываться к формату
Analysis Details0 Issues
Coverage and Duplications
Project ID: PSSL |
ivanmolodec
deleted the
fix-bug-format-in-out-messages-at-sftp-history-integration
branch
Closes #185
Summary by CodeRabbit