#829 proxy support

Слито
KitaitiMakoto слито 2 коммит(ов) из dr-bonez/Plume:feature/proxy в main 4 дней назад
dr-bonez прокомментировал 3 месяцев назад

Adds support for using a proxy for federation.
Motivation is for being able to support homeservers running as Tor hidden services.

Threading config information into plume-common is the only reason this pr isn’t very low impact.

Adds support for using a proxy for federation. Motivation is for being able to support homeservers running as Tor hidden services. Threading config information into plume-common is the only reason this pr isn't very low impact.
KitaitiMakoto прокомментировал 1 месяц назад
Владелец

Thank you for pull request and sorry for late reply. But we have less time to review, now. Can you wait for a while, please? Thanks.

Thank you for pull request and sorry for late reply. But we have less time to review, now. Can you wait for a while, please? Thanks.
KitaitiMakoto прокомментировал 2 недель назад
Владелец

Sorry for very late reply.

What need proxy config is only ClientBuilder. How about to use CONFIG.proxy() directly in plume-common/src/activity_pub/mod.rs?

Sorry for very late reply. What need proxy config is only `ClientBuilder`. How about to use `CONFIG.proxy()` directly in `plume-common/src/activity_pub/mod.rs`?
dr-bonez прокомментировал 6 дней назад
Автор

I can’t. CONFIG comes from plume-models, which has plume-common as a dependency. Pretty sure you can’t have 2 crates that depend on each other.

I can't. CONFIG comes from plume-models, which has plume-common as a dependency. Pretty sure you can't have 2 crates that depend on each other.
dr-bonez прокомментировал 6 дней назад
Автор

rebased

rebased
KitaitiMakoto одобрил(а) эти изменения 4 дней назад
KitaitiMakoto прокомментировал 4 дней назад
Владелец

Okay, I will merge. Thank you for your work!

Okay, I will merge. Thank you for your work!
KitaitiMakoto прокомментировал 4 дней назад
Владелец

Okay, I will merge. Thank you for your work!

Okay, I will merge. Thank you for your work!
KitaitiMakoto Объединил коммит ec8c17482d на main 4 дней назад

Рецензенты

KitaitiMakoto одобрил(а) эти изменения 4 дней назад
Pull Request был объединен как %!с(string=ec8c17482d).
Войдите, чтобы присоединиться к обсуждению.
Нет рецензентов
Нет этапа
Нет назначенных лиц
2 участников
Уведомления
Срок выполнения

Срок выполнения не установлен.

Зависимости

Этот запрос на слияние в настоящее время не имеет никаких зависимостей.

Загрузка…
Пока нет содержимого.