#829 proxy support

Fusionnée
KitaitiMakoto a fusionné 2 révision(s) à partir de dr-bonez/Plume:feature/proxy vers main il y a 4 jours

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 a commenté il y a 1 mois
Propriétaire

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 a commenté il y a 2 semaines
Propriétaire

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 a commenté il y a 6 jours
Publier

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 a commenté il y a 6 jours
Publier

rebased

rebased
KitaitiMakoto ces changements ont été approuvés il y a 4 jours
KitaitiMakoto a commenté il y a 4 jours
Propriétaire

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto a commenté il y a 4 jours
Propriétaire

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto commit de fusion ec8c17482d dans main il y a 4 jours

Relecteurs

KitaitiMakoto ces changements ont été approuvés il y a 4 jours
La demande de tirage a été fusionnée en ec8c17482d.
Connectez-vous pour rejoindre cette conversation.
Pas de relecteur
Aucun jalon
Pas d'assignataires
2 participants
Notifications
Échéance

Aucune échéance n'a été définie.

Dépendances

La demande de fusion n'a actuellement pas de dépendance.

Chargement…
Il n'existe pas encore de contenu.