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

Баг скачивания пакетов в 0.9.5 #70

Closed
artbear opened this issue May 3, 2017 · 3 comments
Closed

Баг скачивания пакетов в 0.9.5 #70

artbear opened this issue May 3, 2017 · 3 comments
Assignees
Labels

Comments

@artbear
Copy link
Member

artbear commented May 3, 2017

http://ci.silverbulleters.org/job/Vanessa-runner/job/develop/lastFailedBuild/console

opm update 1bdd 

20:00:21 КРИТИЧНАЯОШИБКА - 
{Модуль C:\Program Files (x86)\OneScript\lib\opm\src\Классы\Зеркало.os / 
Ошибка в строке: 16 / Преобразование к типу 'Число' не поддерживается}

Проблема в строке https://github.com/oscript-library/opm/blob/master/src/%D0%9A%D0%BB%D0%B0%D1%81%D1%81%D1%8B/%D0%97%D0%B5%D1%80%D0%BA%D0%B0%D0%BB%D0%BE.os#L15

Копипаста присутствует, отличия в коде проверки условий Если НастройкиПрокси.ХХХ или Если НастройкиПрокси.ХХХ = Ложь/Истина

В Зеркало.ос

Функция ИнициализироватьСоединение() Экспорт

	Если Не Соединение = Неопределено Тогда
		Возврат Соединение;
	КонецЕсли;
	
	НастройкиПрокси = НастройкиПриложенияOpm.Получить().Прокси;
	Если НастройкиПрокси.ИспользоватьПрокси Тогда
		Прокси = Новый ИнтернетПрокси(НастройкиПрокси.ПроксиПоУмолчанию);
		Если НастройкиПрокси.ПроксиПоУмолчанию Тогда
			Прокси.Установить("http",НастройкиПрокси.Сервер,НастройкиПрокси.Порт,НастройкиПрокси.Пользователь,НастройкиПрокси.Пароль,НастройкиПрокси.ИспользоватьАутентификациюОС);
		КонецЕсли;	
		Соединение = Новый HTTPСоединение(СерверУдаленногоХранилища,,,,Прокси);
	Иначе
		Соединение = Новый HTTPСоединение(СерверУдаленногоХранилища);
	КонецЕсли;
	
	Возврат Соединение;
	
КонецФункции

В УстановкаПакета.ос

Функция ИнициализироватьСоединение(Сервер) Экспорт
	
	НастройкиПрокси = НастройкиПриложенияOpm.Получить().Прокси;
	Если НастройкиПрокси.ИспользоватьПрокси = Истина Тогда
		Прокси = Новый ИнтернетПрокси(НастройкиПрокси.ПроксиПоУмолчанию);
		Если НастройкиПрокси.ПроксиПоУмолчанию = Ложь Тогда
			Прокси.Установить("http",НастройкиПрокси.Сервер,НастройкиПрокси.Порт,НастройкиПрокси.Пользователь,НастройкиПрокси.Пароль,НастройкиПрокси.ИспользоватьАутентификациюОС);
		КонецЕсли;	
		Соединение = Новый HTTPСоединение(Сервер,,,,Прокси);
	Иначе
		Соединение = Новый HTTPСоединение(Сервер);
	КонецЕсли;
	
	Возврат Соединение;
	
КонецФункции	
@artbear artbear added the bug label May 3, 2017
@artbear artbear added this to the 0.10 milestone May 3, 2017
@artbear
Copy link
Member Author

artbear commented May 3, 2017

Но, возможно, проблема в следующей строке Прокси.Установить("http",НастройкиПрокси.Сервер,НастройкиПрокси.Порт,НастройкиПрокси.Пользователь,НастройкиПрокси.Пароль,НастройкиПрокси.ИспользоватьАутентификациюОС);

проблема на CI, ручной запуск на личной машине ошибку не показал

@artbear
Copy link
Member Author

artbear commented May 3, 2017

@nixel2007 @dmpas Проблема еще веселей.
В очередной раз доказано, что копипаста зло :)
В УстановкаПакета написано Если НастройкиПрокси.ПроксиПоУмолчанию = Ложь
А в Зеркало написано обратное условие Если НастройкиПрокси.ПроксиПоУмолчанию, а остальной код совпадает :(

@artbear
Copy link
Member Author

artbear commented May 3, 2017

Да, проблема была в неверном условии.
Исправлено.

@artbear artbear closed this as completed May 3, 2017
@artbear artbear removed this from the 1.0.0 milestone Jan 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant