#829 proxy support

Merge aplicado

KitaitiMakoto aplicou merge dos 2 commits de dr-bonez/Plume:feature/proxy em main 4 dias atrás

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 aprovou estas alterações 4 dias atrás

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

KitaitiMakoto citou esta issue em um commit 4 dias atrás

KitaitiMakoto merged commit ec8c17482d into main 4 dias atrás

KitaitiMakoto aprovou estas alterações 4 dias atrás

O pull request teve merge aplicado como ec8c17482d.

Acesse para participar desta conversação.

KitaitiMakoto

Sem etiqueta

Sem marco

Sem responsável

2 participante(s)

Notificações

Data limite

Data limite não informada.

Dependências

Atualmente este pull request não tem dependências.