#829 proxy support

Zusammengeführt
KitaitiMakoto hat 2 Commits von dr-bonez/Plume:feature/proxy nach main vor 4 Tagen zusammengeführt
dr-bonez hat vor 3 Monaten kommentiert

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 hat vor 1 Monat kommentiert
Besitzer

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 hat vor 2 Wochen kommentiert
Besitzer

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 hat vor 6 Tagen kommentiert
Ersteller

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 hat vor 6 Tagen kommentiert
Ersteller

rebased

rebased
KitaitiMakoto hat die Änderungen vor 4 Tagen genehmigt
KitaitiMakoto hat vor 4 Tagen kommentiert
Besitzer

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto hat vor 4 Tagen kommentiert
Besitzer

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto hat den Commit ec8c17482d in main vor 4 Tagen gemergt

Reviewer

KitaitiMakoto hat die Änderungen vor 4 Tagen genehmigt
Der Pull Request wurde als ec8c17482d zusammengeführt.
Anmelden, um an der Diskussion teilzunehmen.
Keine Reviewer
Kein Meilenstein
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Dieser Pull-Request hat momentan keine Abhängigkeiten.

Laden…
Hier gibt es bis jetzt noch keinen Inhalt.