proxy support #829

手動マージ済み
KitaitiMakoto が 2 個のコミットを dr-bonez/Plume:feature/proxy から main へマージ 2021-01-13 14:20:37 +00:00
貢献者

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.
オーナー

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.
オーナー

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`?
作成者
貢献者

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.
作成者
貢献者

rebased

rebased
KitaitiMakoto が変更を承認 2021-01-13 14:16:23 +00:00
オーナー

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

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

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

Okay, I will merge. Thank you for your work!
KitaitiMakoto がコミット ec8c17482dmain に手動マージ 2021-01-13 14:20:37 +00:00
サインインしてこの会話に参加。
レビューアなし
マイルストーンなし
プロジェクトなし
担当者なし
2 人の参加者
通知
期日
期日が正しくないか範囲を超えています。 "yyyy-mm-dd" の形式で入力してください。

期日は未設定です。

依存関係

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

リファレンス: Plume/Plume#829
説明はありません。