#829 proxy support

Birleştirildi
KitaitiMakoto 4 gün önce dr-bonez/Plume:feature/proxy içindeki 2 işleme main ile birleştirdi
dr-bonez 3 ay önce yorum yaptı

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 1 ay önce yorum yaptı
Sahibi

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 2 hafta önce yorum yaptı
Sahibi

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 6 gün önce yorum yaptı
Poster

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 6 gün önce yorum yaptı
Poster

rebased

rebased
KitaitiMakoto 4 gün önce bu değişiklikleri onayladı
KitaitiMakoto 4 gün önce yorum yaptı
Sahibi

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto 4 gün önce yorum yaptı
Sahibi

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto ec8c17482d işlemesini main 4 gün önce ile birleştirdi

Gözden Geçirenler

KitaitiMakoto 4 gün önce bu değişiklikleri onayladı
Değişiklik isteği ec8c17482d olarak birleştirildi.
Bu konuşmaya katılmak için oturum aç.
Değerlendirici yok
Kilometre Taşı Yok
Atanan Kişi Yok
2 Katılımcı
Bildirimler
Bitiş Tarihi

Bitiş tarihi atanmadı.

Bağımlılıklar

Bu çekme isteği henüz bir bağımlılık içermiyor.

Yükleniyor…
Henüz bir içerik yok.