Skip to content
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

feat: allow config service only on client #275

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

feat: allow config service only on client #275

wants to merge 4 commits into from

Conversation

yorktown-class
Copy link

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"

@rapiz1
Copy link
Owner

rapiz1 commented Oct 1, 2023

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

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

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
@yorktown-class @rapiz1 @ZHLHZHU