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

gNMIc REST API interface protection #529

Open
vseregin63 opened this issue Oct 10, 2024 · 3 comments
Open

gNMIc REST API interface protection #529

vseregin63 opened this issue Oct 10, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@vseregin63
Copy link

vseregin63 commented Oct 10, 2024

Hi, Team!

We use gnmic cluster of two nodes. In such a cluster master node assign targets to slave via http rest api. Is where is a way to protect rest api interfaces of nodes from another network connection? May be user/pass auth, token auth or tls certs auth are supported on rest api? Please direct to docs.

Thnx!

@vseregin63 vseregin63 changed the title GNMIC REST API interface protection gNMIc REST API interface protection Oct 10, 2024
@vseregin63
Copy link
Author

vseregin63 commented Oct 22, 2024

Have a good day!

I found in docs how to configure api server with tls auth required (https://gnmic.openconfig.net/user_guide/api/api_intro/) and added rootCA and server key/cert to gnmic config file

api-server:
tls:
ca-file: /var/run/secrets/rootCA.crt
cert-file: /var/run/secrets/gnmic_api_server.pem
key-file: /var/run/secrets/gnmic_api_server.key
client-auth: "require-verify"

It works, but i still do not understand where to put generated client cert and key bacouse they needed for protected intra cluster operation. Without it leader tries to assign element to slave gnmic via https and fails (doesn`t have a cert)

2024/10/22 12:36:38.240084 [gnmic] failed assigning target "leaf1" to service "gnmic-api": Post "https://server.com:7890/api/v1/config/targets": remote error: tls: certificate required

@karimra Please help

@karimra
Copy link
Collaborator

karimra commented Oct 23, 2024

mTLS is currently not supported by the clustering client. I agree that it will be useful, I will look into it.

@karimra karimra added the enhancement New feature or request label Oct 23, 2024
@vseregin63
Copy link
Author

Thanks for reply, @karimra! If you could implement this, it will be great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants