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

Authentication example #275

Open
bgokden opened this issue Feb 10, 2020 · 5 comments
Open

Authentication example #275

bgokden opened this issue Feb 10, 2020 · 5 comments
Assignees
Labels
help wanted Extra attention is needed question Further information is requested

Comments

@bgokden
Copy link

bgokden commented Feb 10, 2020

Hello, I am looking for an example where two nodes authenticate with a pre-shared key.
Is it possible or should I add a new extension?

@iwasaki-kenta iwasaki-kenta added help wanted Extra attention is needed question Further information is requested labels Feb 10, 2020
@iwasaki-kenta iwasaki-kenta self-assigned this Feb 10, 2020
@iwasaki-kenta
Copy link
Contributor

It is not possible just yet to authenticate with a pre-shared key, though exposing the option to provide a pre-shared key should be straightforward. It would involve modifying the handshake function in client.go.

However, I currently am busy right now with other projects, unfortunately. I will only be able to integrate in this option around next week. However, if you need it and able are to tackle it now, I would appreciate a PR! Happy to help out should that be the case too :).

@haukened
Copy link

haukened commented Jul 7, 2020

any update on this? we're starting a project with Noise and just identified a need for this today, to prevent a "poisoned peer" situation where anyone running the noise protocol could attempt to talk to your peer network.

@bgokden
Copy link
Author

bgokden commented Jul 7, 2020

@haukened I actually don't have a working solution for this. I have thought of two solutions.
The first one was editing the first message exchange.
The second one was adding a new protocol that looked nicer solution, but I couldn't finish it and I am not working on it anymore.

@haukened
Copy link

haukened commented Jul 7, 2020

Thanks for the update bgkoden. i think adding a protocol might be what i need here! I'll likely extend gossip to have a PSK field.

@danthegoodman1
Copy link

@haukened What did you end up doing? I am working on something with a similar need.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants