proxy support #829

手動マージ済み

KitaitiMakoto が 2 個のコミットを dr-bonez/Plume:feature/proxy から main へマージ

dr-bonez がコメント

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 がコメント

オーナー

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 がコメント

オーナー

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 がコメント

投稿者

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 がコメント

投稿者

rebased

KitaitiMakoto が変更を承認

KitaitiMakoto がコメント

オーナー

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

KitaitiMakoto がコメント

オーナー

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

KitaitiMakoto がコミットでこのイシューを参照

Merge pull request 'proxy support' (#829) from dr-bonez/Plume:feature/proxy into main Reviewed-on: https://git.joinplu.me/Plume/Plume/pulls/829 Reviewed-by: KitaitiMakoto <kitaitimakoto@noreply@joinplu.me>

KitaitiMakoto がコミット ec8c17482d を main に手動マージ 3年前

レビューア

KitaitiMakoto が変更を承認

プルリクエストは ec8c17482d で手動マージされています。

サインインしてこの会話に参加。

レビューアなし

マイルストーンなし

担当者なし

2 人の参加者

通知

期日

期日が正しくないか範囲を超えています。 'yyyy-mm-dd' の形式で入力してください。

期日は未設定です。

依存関係

依存関係が設定されていません。

読み込み中…

まだ内容がありません

ラベル マイルストーン

proxy support #829

レビューア

ラベルマイルストーン