#829 proxy support

병합

KitaitiMakoto dr-bonez/Plume:feature/proxy 에서 main 로 2 commits 를 머지했습니다 4 일 전

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.

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.

rebased

KitaitiMakoto 이 변경사항을 승인하였습니다. 4 일 전

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

KitaitiMakoto 커밋 4 일 전에서 이 이슈 언급

KitaitiMakoto merged commit ec8c17482d into main 4 일 전

KitaitiMakoto 이 변경사항을 승인하였습니다. 4 일 전

The pull request has been merged as ec8c17482d.

로그인하여 이 대화에 참여

KitaitiMakoto

레이블 없음

마일스톤 없음

담당자 없음

참여자 2명

알림

마감일

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

의존성

이 풀 리퀘스트는 어떠한 의존성도 가지지 않습니다.