#829 proxy support

Integração executada
KitaitiMakoto integrou 2 cometimentos de %!<(string=dr-bonez/Plume:feature/proxy)/code> em main há 4 dias
dr-bonez comentado há 3 meses

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 comentado há 1 mês
Proprietário(a)

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 comentado há 2 semanas
Proprietário(a)

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 comentado há 6 dias
Autor

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 comentado há 6 dias
Autor

rebased

rebased
KitaitiMakoto aprovou estas alterações há 4 dias
KitaitiMakoto comentado há 4 dias
Proprietário(a)

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto comentado há 4 dias
Proprietário(a)

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto cometimento ec8c17482d integrado em main há 4 dias

Revisores

KitaitiMakoto aprovou estas alterações há 4 dias
A integração constante no pedido foi executada como ec8c17482d.
Inicie a sessão para participar neste diálogo.
Sem revisores
Sem etapa
Sem responsáveis
2 Participantes
Notificações
Date limite

Sem data limite definida.

Dependências

Este pedido de integração não tem quaisquer dependências, neste momento.

Carregando…
Ainda não há conteúdo.