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.

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

Add ability to specify grpc host #18193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cl-bvl
Copy link

@cl-bvl cl-bvl commented Feb 18, 2025

Hello,
Please excuse any potential bugs, as I'm not a C++ developer. This update adds the ability to specify the gRPC listening host. Listening on 0.0.0.0 by default is not secure and could lead to significant security breaches due to firewall misconfigurations.

Rest assured, the default behavior remains unchanged for existing users.

@cl-bvl cl-bvl force-pushed the feature/grpc_listen_unix_or_host branch from 0a11e3d to 350ac60 Compare February 18, 2025 09:37
@frrbot frrbot bot added bgp bugfix tests Topotests, make check, etc yang labels Feb 18, 2025
@github-actions github-actions bot added size/L and removed size/M labels Feb 18, 2025
@cl-bvl cl-bvl force-pushed the feature/grpc_listen_unix_or_host branch from 350ac60 to b707e20 Compare February 18, 2025 09:38
@github-actions github-actions bot added size/M and removed size/L labels Feb 18, 2025
@donaldsharp
Copy link
Member

a) This branch is clearly behind because both Mark's and Carmine's code is already in the tree and has been for a couple of days. Please rebase to latest master and force push.
b) You should also remove the merge commit we do not want that to come in as part of your code
c) There is absolutely no documentation for this change at all. Please update the docs to reflect what is going on and how to use this change
d) The grpc commit itself has absolutely no discussion about the what/why/how. Just stating `doing X' is insufficient. Please add some discussion.

@cl-bvl cl-bvl force-pushed the feature/grpc_listen_unix_or_host branch from a128372 to 7ebb5df Compare February 19, 2025 15:05
@github-actions github-actions bot added size/L and removed size/M labels Feb 19, 2025
This update lets you specify the network interface for gRPC.
For example, using 127.0.0.1 restricts gRPC access to the local machine, enhancing security since gRPC is not publicly exposed.
For backward compatibility, the default remains 0.0.0.0.

Signed-off-by: Vladimir Buyanov <[email protected]>
@cl-bvl cl-bvl force-pushed the feature/grpc_listen_unix_or_host branch from 7ebb5df to 45e10c7 Compare February 19, 2025 15:23
@github-actions github-actions bot added size/M and removed size/L labels Feb 19, 2025
@cl-bvl
Copy link
Author

cl-bvl commented Feb 19, 2025

Thanks for comments, done

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

Successfully merging this pull request may close these issues.

2 participants