proxy support #829

Manually merged
KitaitiMakoto 님이 dr-bonez/Plume:feature/proxy 에서 main 로 2 커밋을 2021-01-13 14:20:37 +00:00 병합함
Contributor

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`?
작성자
Contributor

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.
작성자
Contributor

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 manually merged commit ec8c17482d into main 2021-01-13 14:20:37 +00:00
로그인하여 이 대화에 참여하세요.
No reviewers
마일스톤 없음
프로젝트 없음
담당자 없음
참가자 2명
알림
마감일
기한이 올바르지 않거나 범위를 벗어났습니다. "yyyy-mm-dd"형식을 사용해주십시오.

마감일이 설정되지 않았습니다.

전제조건

전제조건이 설정되지 않았습니다.

Reference: Plume/Plume#829
No description provided.