По большей части фикс бесполезных срабатываний sonar

vbondarevsky · Nov 22, 2020 · b8f2cb7 · b8f2cb7
1 parent 3365204
commit b8f2cb7
Show file tree

Hide file tree

Showing 9 changed files with 55 additions and 40 deletions.
diff --git a/.bsl-language-server.json b/.bsl-language-server.json
@@ -2,7 +2,7 @@
     "$schema": "https://1c-syntax.github.io/bsl-language-server/configuration/schema.json",
     "diagnostics": {
         "Typo": {
-            "userWordsToIgnore": "Редирект,Редиректа,Распарсить,Сериализации"
+            "userWordsToIgnore": "Редирект,Редиректа,Сериализации"
         }
     }
 }
diff --git a/src/internal/Классы/ПарсерURL.os b/src/internal/Классы/ПарсерURL.os
@@ -1,10 +1,12 @@
-Перем Схема Экспорт;
-Перем Аутентификация Экспорт;
-Перем Сервер Экспорт;
-Перем Порт Экспорт;
-Перем Путь Экспорт;
-Перем ПараметрыЗапроса Экспорт;
-Перем Фрагмент Экспорт;
+// BSLLS:ExportVariables-off
+Перем Схема Экспорт; // Схема
+Перем Аутентификация Экспорт; // Параметры аутентификации
+Перем Сервер Экспорт; // Сервер
+Перем Порт Экспорт; // Порт
+Перем Путь Экспорт; // Путь на сервере
+Перем ПараметрыЗапроса Экспорт; // Часть адреса после ?
+Перем Фрагмент Экспорт; // Часть адреса после #
+// BSLLS:ExportVariables-on
 
 Процедура ПриСозданииОбъекта(Знач URL)
 

diff --git a/src/internal/Классы/ПулСоединений.os b/src/internal/Классы/ПулСоединений.os
@@ -1,5 +1,7 @@
-Перем ПулСоединений;
-Перем МаксимальноеВремяПростоя;
+// BSLLS:ExportVariables-off
+Перем ПулСоединений; // Объект для хранения пула соединений
+Перем МаксимальноеВремяПростоя; // Время неиспользования соединения, после которого оно удаляется из пула
+// BSLLS:ExportVariables-on
 
 Процедура ПриСозданииОбъекта()
 

diff --git a/src/internal/Модули/КоннекторHTTPСлужебный.os b/src/internal/Модули/КоннекторHTTPСлужебный.os
@@ -1,4 +1,6 @@
-Перем мПулСоединений Экспорт;
+// BSLLS:ExportVariables-off
+Перем мПулСоединений Экспорт; // Служебная переменная для хранения пула соединений
+// BSLLS:ExportVariables-on
 
 Функция ВызватьHTTPМетод(Сессия, Метод, URL, ДополнительныеПараметры) Экспорт
 
@@ -750,7 +752,7 @@
 	Для Каждого ОчереднойЗаголовок Из Заголовки Цикл
 		Если НРег(ОчереднойЗаголовок.Ключ) = "set-cookie" Тогда
 			Для Каждого ЗаголовокCookie Из РазбитьНаОтдельныеЗаголовкиCookies(ОчереднойЗаголовок.Значение) Цикл
-				Cookie = РаспарситьCookie(ЗаголовокCookie, URL, ТекущееВремя);
+				Cookie = РазобратьCookie(ЗаголовокCookie, URL, ТекущееВремя);
 				Если Cookie = Неопределено Тогда
 					Продолжить;
 				КонецЕсли;
@@ -821,7 +823,7 @@
 
 КонецФункции
 
-Функция РаспарситьCookie(Заголовок, URL, ТекущееВремя)
+Функция РазобратьCookie(Заголовок, URL, ТекущееВремя)
 
 	Cookie = Неопределено;
 	Индекс = 0;

diff --git a/src/Классы/ПодготовленныйЗапрос.os b/src/Классы/ПодготовленныйЗапрос.os
@@ -1,13 +1,15 @@
 #Использовать "../internal"
 
-Перем Cookies Экспорт;
-Перем Аутентификация Экспорт;
-Перем Метод Экспорт;
-Перем Заголовки Экспорт;
-Перем ПараметрыЗапроса Экспорт;
-Перем URL Экспорт;
-Перем ПараметрыПреобразованияJSON Экспорт;
-Перем HTTPЗапрос Экспорт;
+// BSLLS:ExportVariables-off
+Перем Cookies Экспорт; // Cookies запроса
+Перем Аутентификация Экспорт; // Параметры аутентификации запроса
+Перем Метод Экспорт; // Метод запроса
+Перем Заголовки Экспорт; // Заголовки запроса
+Перем ПараметрыЗапроса Экспорт; // Параметры запроса
+Перем URL Экспорт; // URL запроса
+Перем ПараметрыПреобразованияJSON Экспорт; // Параметры преобразования JSON запроса
+Перем HTTPЗапрос Экспорт; // Заполненный объект HTTPЗапрос
+// BSLLS:ExportVariables-on
 
 Процедура ПриСозданииОбъекта(Сессия, МетодHTTP, СтрокаURL, ДополнительныеПараметры)
 

diff --git a/src/Классы/ПодготовленныйОтвет.os b/src/Классы/ПодготовленныйОтвет.os
@@ -1,15 +1,18 @@
 #Использовать "../internal"
 
-Перем ВремяВыполнения Экспорт;
-Перем Cookies Экспорт;
-Перем Заголовки Экспорт;
-Перем ЭтоПостоянныйРедирект Экспорт;
-Перем ЭтоРедирект Экспорт;
-Перем Кодировка Экспорт;
-Перем КодСостояния Экспорт;
-Перем URL Экспорт;
-Перем мИсходныйОтвет;
-Перем мПараметрыПреобразованияJSON;
+// BSLLS:ExportVariables-off
+Перем ВремяВыполнения Экспорт; // Время выполнения запроса
+Перем Cookies Экспорт; // Полученные Cookies
+Перем Заголовки Экспорт; // Заголовки ответа
+Перем ЭтоПостоянныйРедирект Экспорт; // Ответ содержит постоянный редирект
+Перем ЭтоРедирект Экспорт; // Ответ содержит редирект
+Перем Кодировка Экспорт; // Кодировка ответа
+Перем КодСостояния Экспорт; // Код состояния ответа
+Перем URL Экспорт; // URL, по которому был запрошен ответ
+// BSLLS:ExportVariables-on
+
+Перем мИсходныйОтвет; // Служебная переменная для хранения объекта HTTPОтвет
+Перем мПараметрыПреобразованияJSON; // Служебная переменная для хранения параметров преобразования JSON
 
 Процедура ПриСозданииОбъекта(ИсходныйОтвет, ПараметрыПреобразованияJSON)
 	мИсходныйОтвет = ИсходныйОтвет;

diff --git a/src/Классы/Сессия.os b/src/Классы/Сессия.os
@@ -1,12 +1,14 @@
 #Использовать "../internal"
 
-Перем Заголовки Экспорт;
-Перем Аутентификация Экспорт;
-Перем Прокси Экспорт;
-Перем ПараметрыЗапроса Экспорт;
-Перем МаксимальноеКоличествоПеренаправлений Экспорт;
-Перем Cookies Экспорт;
-Перем СлужебныеДанные Экспорт;
+// BSLLS:ExportVariables-off
+Перем Заголовки Экспорт; // Заголовки сессии
+Перем Аутентификация Экспорт; // Параметры аутентификации сессии
+Перем Прокси Экспорт; // ИнтернетПрокси сессии
+Перем ПараметрыЗапроса Экспорт; // Параметры запроса сессии
+Перем МаксимальноеКоличествоПеренаправлений Экспорт; // Максимальное количество перенаправлений
+Перем Cookies Экспорт; // Cookies сессии
+Перем СлужебныеДанные Экспорт; // Служебные данные сессии
+// BSLLS:ExportVariables-on
 
 Процедура ПриСозданииОбъекта()
 

diff --git a/src/Модули/КодыСостоянияHTTP.os b/src/Модули/КодыСостоянияHTTP.os
@@ -1,3 +1,5 @@
+// BSLLS:MissingVariablesDescription-off
+// BSLLS:ExportVariables-off
 Перем ПеремещеноНавсегда_301 Экспорт;
 Перем ПеремещеноВременно_302 Экспорт;
 Перем СмотретьДругое_303 Экспорт;
@@ -10,7 +12,8 @@
 Перем ОшибочныйШлюз_502 Экспорт;
 Перем СервисНедоступен_503 Экспорт;
 Перем ШлюзНеОтвечает_504 Экспорт;
-
+// BSLLS:ExportVariables-on
+// BSLLS:MissingVariablesDescription-on
 
 ПеремещеноНавсегда_301 = 301;
 ПеремещеноВременно_302 = 302;

diff --git a/tests/hmac-test.os b/tests/hmac-test.os
@@ -47,7 +47,6 @@
 
 КонецПроцедуры
 
-
 Функция Ключ()
 
 	Возврат ПолучитьДвоичныеДанныеИзСтроки("Секретный ключ", КодировкаТекста.UTF8, Ложь);