Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NEW] У web- и http-сервиса неверно задан обработчик метода #1222

Closed
5 of 24 tasks
artbear opened this issue May 28, 2020 · 5 comments · Fixed by #1792
Closed
5 of 24 tasks
Labels
component/diagnostics Доработка / создание диагностики good first issue Good for newcomers

Comments

@artbear
Copy link
Contributor

artbear commented May 28, 2020

Описание проблемы, ошибки, которую надо диагностировать

Разработчики иногда допускают ошибки при создании множества методов веб-сервисов, забывая указать метод-обработчик веб-сервиса.

Конфигуратор ругается только при включенном флаге "Проверка существования назначенных обработчиков"

Ссылка на источник, подтверждающее нарушение либо обоснование наличия проблемы

Параметры диагностики

Тип Статья на русском

  • 🐜 Ошибка
  • 👮 Уязвимость
  • 💂‍♂️ Потенциальная уязвимость
  • 💩 Качество кода
  • :trollface: Другое

Важность Статья на русском

  • 💔 Блокирующая / Blocker
  • ❤️ Критическая / Critical
  • 💛 Важная / Major
  • 💙 Незначительная / Minor
  • 💚 Информационная / Info
  • 💞 Другое

Тэги Статья на русском

  • STANDARD - "Нарушение стандартов 1С"
  • LOCKINOS - "Не будет работать в другой ОС"
  • SQL - "Проблема с запросом"
  • PERFORMANCE - "Проблема производительности"
  • BRAINOVERLOAD - "Непонятный код"
  • BADPRACTICE - "Плохая практика программирования"
  • CLUMSY - "Излишние действия"
  • DESIGN - "Ошибка в проектировании"
  • SUSPICIOUS - "Подозрительный код"
  • UNPREDICTABLE - "Непредсказуемо работающий код"
  • DEPRECATED - "Устаревшая функциональность"
  • ERROR - "Ошибочная конструкция"
  • LOCALIZE - "Проблемы локализации"

Время на исправление (минут)

10 минут

Дополнительная информация

@artbear artbear added the component/diagnostics Доработка / создание диагностики label May 28, 2020
@theshadowco theshadowco added the require/metadata Требуются метаданные label Jun 3, 2020
@zeegin
Copy link
Contributor

zeegin commented Jun 5, 2020

Предлагаю повысить до Блокирующий и Ошибка

@artbear
Copy link
Contributor Author

artbear commented Aug 14, 2021

@theshadowco Уже можно реализовывать правило, т.к. поддержка метаданных реализована в связанном ишузе, верно?
тогда сними тег require/metadata

@theshadowco
Copy link
Member

В операциях вебсервисов и хттп сервисов есть свойство handler

@theshadowco theshadowco added the good first issue Good for newcomers label Aug 17, 2021
@artbear artbear changed the title [NEW] У web- и http-сервиса не задан обработчик события [NEW] У web- и http-сервиса не задан обработчик метода Sep 4, 2021
@artbear
Copy link
Contributor Author

artbear commented Sep 4, 2021

@nixel2007 Назначь на меня, я большую часть правил уже сделал.

@artbear artbear changed the title [NEW] У web- и http-сервиса не задан обработчик метода [NEW] У web- и http-сервиса неверно задан обработчик метода Sep 5, 2021
@artbear
Copy link
Contributor Author

artbear commented Sep 5, 2021

@zeegin @nixel2007 Вопрос - Нужно ли в этом правиле проверять наличие хотя бы одного возврата в методе-обработчике?

правило на наличие Возвратов есть, но у него важность более низкая, и замечания по нему не являются ошибкой.
а для сервисов важно явно что-то вернуть вместо неявного возврата Неопределено, если нет возвратов.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/diagnostics Доработка / создание диагностики good first issue Good for newcomers
Projects
None yet
3 participants