Skip to content

Conversation

asosnoviy
Copy link
Member

Исправлен протокол и прокси.
Добавлен параметр ИспользоватьАутентификациюОС в файл конфига
Проверил на каком то странном прокси-сервере под win "ccproxy"

@artbear
Copy link
Member

artbear commented Nov 17, 2016

T:\>opm install dummy
Свойство объекта не обнаружено (ИспользоватьАутентификациюОС)

Баг :(
Тесты, тесты где ? :) :(

@artbear artbear self-assigned this Nov 17, 2016
@asosnoviy
Copy link
Member Author

Обязуюсь без тестов ничего больше не пулить! opm.cfg не от старой версии остался?

@artbear
Copy link
Member

artbear commented Nov 17, 2016

Да, конфигурация старая :(
удаляю, баг пропадает.
Нужно в файл добавить ИспользоватьАутентификациюОС, верно?

@artbear
Copy link
Member

artbear commented Nov 17, 2016

Ручное тестирование показало, что прокси заработало

Без настроенного прокси выдается 407 ошибка

T:\> opm install mess
ИНФОРМАЦИЯ - Скачиваю файл: mess.ospx
Ошибка установки пакета mess <407>

С правильно настроенным прокси выдаются правильные коды и все устанавливается

T:\>opm install mess
ИНФОРМАЦИЯ - Скачиваю файл: mess.ospx
Ошибка установки пакета mess <404>

T:\>opm install messenger
ИНФОРМАЦИЯ - Скачиваю файл: messenger.ospx
ИНФОРМАЦИЯ - Устанавливаю пакет messenger
ИНФОРМАЦИЯ - Регистрация приложения: messenger
ИНФОРМАЦИЯ - Установка завершена

@artbear
Copy link
Member

artbear commented Nov 17, 2016

на системе без прокси проверить не могу.
@asosnoviy ты проверял работу на системе, где нет прокси?

@asosnoviy
Copy link
Member Author

@artbear да проверял.
Вот сейчас еще раз перепроверю.


с "прямым" интернетом

Когда я устанавливаю свежий opm
И Выполняю opm update messenger
Тогда Не создается файл Opm.cfg
И результат выполнения команды

ИНФОРМАЦИЯ - Скачиваю файл: messenger.ospx
ИНФОРМАЦИЯ - Устанавливаю пакет messenger
ИНФОРМАЦИЯ - Регистрация приложения: messenger
ИНФОРМАЦИЯ - Установка завершена


Когда я выполняю opm config тогда
Создается файл opm.cfg
И в нем установлены параметры

Параметр Значение
ИспользоватьПрокси false
ПроксиПоУмолчанию false
Сервер ""
Порт ""
Пользователь ""
Пароль ""
ИспользоватьАутентификациюОС false

Когда Выполняю opm update messenger
Тогда результат выполнения команды

ИНФОРМАЦИЯ - Скачиваю файл: messenger.ospx
ИНФОРМАЦИЯ - Устанавливаю пакет messenger
ИНФОРМАЦИЯ - Регистрация приложения: messenger
ИНФОРМАЦИЯ - Установка завершена


Когда я выполняю opm config -usedefaultproxy
Тогда значения в файле стали равны

Параметр Значение
ИспользоватьПрокси true
ПроксиПоУмолчанию true
Сервер ""
Порт ""
Пользователь ""
Пароль ""
ИспользоватьАутентификациюОС false

Когда Выполняю opm update messenger
Тогда результат выполнения команды

ИНФОРМАЦИЯ - Скачиваю файл: messenger.ospx
ИНФОРМАЦИЯ - Устанавливаю пакет messenger
ИНФОРМАЦИЯ - Регистрация приложения: messenger
ИНФОРМАЦИЯ - Установка завершена



Когда я выполняю opm config -proxyserver 123.123.123.123 -proxyport 123 -proxyuser test -proxypass password
Тогда значения в файле стали равны

Параметр Значение
ИспользоватьПрокси true
ПроксиПоУмолчанию false
Сервер "123.123.123.123"
Порт "123"
Пользователь "test"
Пароль "password"
ИспользоватьАутентификациюОС false

Когда Выполняю opm update messenger
Тогда результат выполнения команды

ИНФОРМАЦИЯ - Скачиваю файл: messenger.ospx
Внешнее исключение (System.Net.WebException): Невозможно соединиться с удаленным сервером


@asosnoviy
Copy link
Member Author

asosnoviy commented Nov 17, 2016

Да, конфигурация старая :(
удаляю, баг пропадает.
Нужно в файл добавить ИспользоватьАутентификациюОС, верно?

Мне казалось запуск opm config еще раз должен его сам добавить... Но нет.
И еще пока писал понял, что у порта тип должен быть число.

@artbear
Copy link
Member

artbear commented Nov 17, 2016

@asosnoviy Можешь свою фичу закинуть в новый feature-файл проекта opm и с помощью моего проекта https://github.com/artbear/1bdd реализовать тестирование указанного поведения :)

@asosnoviy
Copy link
Member Author

@artbear так и сделаю =)

@asosnoviy
Copy link
Member Author

asosnoviy commented Nov 17, 2016

Вопрос:
Что должно произойти с конфигом если сначала выполнить
opm config -proxyserver 123.123.123.123 -proxyport 123 -proxyuser test -proxypass password
а следом выполнить opm config ?
Сейчас ничего не произойдет. конфиг останется в "инициализированном для прокси" состоянии
a если выполнить opm config -proxyport 321
Тогда поменяется только порт

@artbear
Copy link
Member

artbear commented Nov 17, 2016

Вопроса не понял.
Или это был риторический вопрос?

чт, 17 Ноя 2016, 20:42 Alexey Sosnoviy notifications@github.com:

Вопрос:
Что должно произойти с конфигом если сначала выполнить

opm config -proxyserver 123.123.123.123 -proxyport 123 -proxyuser test
-proxypass password

а после opm config ?
Сейчас ничего не произойдет.
a если выполнить opm config -proxyport 321
Тогда поменяется только порт


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#7 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACyRcXKvButH2-2gcLLbOr4rJxOaeWjCks5q_JIPgaJpZM4K1ZHI
.

@nixel2007
Copy link
Member

Добавить что-то вроде opm config set-default?

@artbear
Copy link
Member

artbear commented Nov 18, 2016

@asosnoviy ответишь на вопрос @nixel2007 ?

Я готов принять PR.

@asosnoviy
Copy link
Member Author

@artbear вопрос от @nixel2007 это ответ на мой вопрос =)
Есть предложение это PR влить в develop и никому не рассказывать, что оно есть =)
Открыть issue в которой обсудить что ожидается от config.
@nixel2007 случайно не проверил работу прокси с этим PR?

@nixel2007
Copy link
Member

@asosnoviy нет, не проверял :(

@EvilBeaver
Copy link
Member

@nixel2007 а есть прокся, чтобы проверить? Проверишь?

@nixel2007
Copy link
Member

@EvilBeaver у меня жестокая прокся была только на прошлой работе, сейчас негде :( скорее у тебя есть такая возможность ;)
До прошлого пулл реквеста у меня даже без прокси не работал опм. :)

@artbear
Copy link
Member

artbear commented Nov 21, 2016

Я проверил на прокси, сначала не работало, затем автор поправил код и
продукт стал верно отрабатывать.

Скорее нужно проверить работу без прокси, ИМХО ее на последних коммитах не
проверяли ;)

вс, 20 Ноя 2016, 19:55 Nikita Gryzlov notifications@github.com:

@EvilBeaver https://github.com/EvilBeaver у меня жестокая прокся была
только на прошлой работе, сейчас негде :( скорее у тебя есть такая
возможность ;)
До прошлого пулл реквеста у меня даже без прокси не работал опм. :)


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#7 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACyRcQyfKoJ-3MCBpPd-4xvnCz8oFO7Yks5rAHuQgaJpZM4K1ZHI
.

@EvilBeaver EvilBeaver merged commit 915dbad into oscript-library:develop Nov 22, 2016
@faberlic-service-1c
Copy link

@nixel2007 Никита, заберешь [текст фичи проверки прокси-поведения]#7 (comment)) в свой репо для фич opm ?
чтобы не забылось

@EvilBeaver
Copy link
Member

@artbear ты чего из-под робота пишешь?

@asosnoviy asosnoviy deleted the opmconfig branch December 6, 2016 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants