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

Security Issue - Private/Public Keypair Exchange for External Clients #1378

Closed
1 task done
DrDoug88 opened this issue Jul 11, 2022 · 4 comments
Closed
1 task done
Assignees
Labels
architecture enhancement New feature or request

Comments

@DrDoug88
Copy link

Contact Details

No response

What happened?

When creating external clients, the Netmaker server will automatically create the public/private keypair on behalf of the client. This is not how the public/private key exchange is supposed to work. The private key should be generated on the client side and only public keys exchanged. I realize that this is done in terms on convenience, but room should be made in order to perform a proper key exchange. Can the UI be updated so that the client's public key can be entered/edited? Also, I see a feature request to allow for Preshared Keys (#1231), can this also be done at the same time? Perhaps even allowing any of the external client configuration details to be edited?

Version

v0.14.5

What OS are you using?

Linux

Relevant log output

No response

Contributing guidelines

  • Yes, I did.
@DrDoug88 DrDoug88 added the bug Something isn't working label Jul 11, 2022
@mattkasun
Copy link
Contributor

Yes, I answered too quickly .... didn't realize you were talking about ext clients

@0xdcarns
Copy link
Contributor

We will look to roadmap this as it would be a fairly straightforward security enhancement

@DrDoug88
Copy link
Author

DrDoug88 commented Jul 11, 2022

Okay great, thank you for the prompt response. I'll keep an eye out for it.

@afeiszli afeiszli added enhancement New feature or request architecture and removed bug Something isn't working labels Jul 18, 2022
@mattkasun
Copy link
Contributor

PR #2100 and PR #2148 implement

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

No branches or pull requests

4 participants