-
Notifications
You must be signed in to change notification settings - Fork 207
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
feature: адаптация библиотеки Коннектор HTTP для работы в клиентском контексте. #51
Conversation
в клиентском контексте. -- методы переведены в контекст НаКлиенте -- для серверных методов, которые содержат вызов метода ХешФункция параметр Алгоритм переведн с системного перечисления на Строку -- добавлена обработка тест Тесты_КоннекторHTTP_Клиент -- упрощена работа с получением настроек прокси
@vbondarevsky Владимир, прошу проверить PR |
А почему в форме, а не в общем модуле?
Редиректы сломались на https://httpbin.org/ |
При встраивании библиотеки во внешние обработки приходится размещать её в модуле формы. Поэтому мне показалось, что такая реализация более удобная для повторного использования. |
Тут либо встраивание в обработку/форму, либо общий модуль, но в расширении или сразу в конфигурации. Сейчас в Коннекторе используется второй вариант. Поэтому предлагаю клиентский код тоже вынести в общий модуль |
-- учтены замечания vbondarevsky#51 (comment) -- библиотека перенесена в общий клиентский модуль КоннекторHTTPКлиент -- методы, для которых обязателен серверный контекст перенесены в модуль КоннекторHTTPВызовСервера -- удалена обработка КоннекторHTTP_Клиент -- изменены тесты
-- Не работает переадресация для URL с указанным портом
@vbondarevsky Владимир, учел ваши замечания
|
Но ведь это все равно не будет работать в браузере, не? |
@zeegin да не будет, но будет работать на стороне клиента. |
Я представляю только один сценарий где может быть полезно - мобильный клиент. |
@zeegin Предложу еще вариант. Внешняя обработка где логика реализована на форме и по этому лазить на сервер за коннектором накладно.. |
Еще есть кейс с доменной авторизацией. |
This reverts commit f613c31.
@vbondarevsky , можно оживить этот pull request ? |
@NkLab присылайте пулл-реквест, это лучшая помощь! |
Задача актуальна, но прошло уже 3 года, работы по PR судя по всему не ведутся, поэтому закрываю. |
Решение проблемы issues #50
Адаптация библиотеки Коннектор HTTP для работы в клиентском контексте (Тонкий клиент)
Выполнено:
Добавлена обработка КоннекторHTTP_Клиент, которая содержит форму модКоннектор в модуле которой реализована библиотека.
Что предстоит сделать: