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] Запрет экспортных глобальных переменных модуля #1043

Closed
6 of 24 tasks
artbear opened this issue Apr 12, 2020 · 4 comments · Fixed by #1232
Closed
6 of 24 tasks
Labels
component/diagnostics Доработка / создание диагностики

Comments

@artbear
Copy link
Contributor

artbear commented Apr 12, 2020

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

В статье с ИТС Использование переменных в программных модулях написано так:

В большинстве случаев, вместо переменных программных модулей следует использовать более подходящие средства разработки платформы 1С:Предприятие. Поскольку область видимости (использования) таких переменных сложно контролировать, то они зачастую становятся источником трудновоспроизводимых ошибок.

Также экспортные переменные модулей могут являться дырами в безопасности.

Диагностика применима к любым модулям, в которых могут быть объявлены экспортные переменные.

В указанной статье с ИТС есть много примеров обхода проблемы.

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

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

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

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

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

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

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

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

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

5 минут

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

@artbear artbear added the component/diagnostics Доработка / создание диагностики label Apr 12, 2020
@alkoleft
Copy link
Contributor

Возьмусь

@nixel2007
Copy link
Member

Может здесь как раз заодно сделать #1208 и построить диагностику на ней?

@nixel2007
Copy link
Member

nixel2007 commented May 27, 2020

Ты как раз хотел немного хардкора

@alkoleft
Copy link
Contributor

alkoleft commented May 27, 2020

Попробую реализовать с прицелом на #1208 , если не трудно, хотелось бы пару примеров существующих проверок подпадающих по #1208

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

Successfully merging a pull request may close this issue.

3 participants