#829 proxy support

Обединени

KitaitiMakoto обедини 2 ревизии от dr-bonez/Plume:feature/proxy във main преди 4 дни

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.

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.

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?

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.

rebased

KitaitiMakoto approved these changes преди 4 дни

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

KitaitiMakoto посочи тази задача от ревизия преди 4 дни

KitaitiMakoto merged commit ec8c17482d into main преди 4 дни

KitaitiMakoto approved these changes преди 4 дни

Тази заявка за сливане е била обединена като ec8c17482d.

Впишете се за да се присъедините към разговора.

KitaitiMakoto

Няма етикет

Няма етап

Няма изпълнители

2 участника

Известия

Due Date

No due date set.

Зависимости

This pull request currently doesn't have any dependencies.