Skip to content

feat: allow config service only on client #275

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
yarshen wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat: allow config service only on client #275

yarshen wants to merge 4 commits into from

Conversation

@yarshen
Copy link

@yarshen yarshen commented Jul 23, 2023

Related to #161

允许server直接使用client对service的配置

在server启用

[server]
accept_client_recommend_service = true

[server.services] #这个东西现在还是必须要有

在client配置

[client.services.echo] 
local_addr = "127.0.0.1:8080" 
recommend_blind_addr = "0.0.0.0:2334"

@yujqiao
Copy link
Member

yujqiao commented Oct 1, 2023

Is it sending the configuration in plain text? That's something unfavored

@yujqiao yujqiao force-pushed the main branch 8 times, most recently from 78f1157 to 97541af Compare October 1, 2023 09:13
@yarshen yarshen force-pushed the feature/config_service_in_client branch from 37f3e79 to ca28e43 Compare October 11, 2023 13:46
@yarshen
Copy link
Author

yarshen commented Oct 11, 2023

I'm sending the configuration in plain text. Won't it be encrypted, even if 'transport.tls' is configured?

@ZHLHZHU
Copy link

ZHLHZHU commented Jan 26, 2024

any progress on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yarshen @yujqiao @ZHLHZHU