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

[Feature]: Add support for the SHA-256 auth algorithm #25

Closed
jwillikers opened this issue Jan 6, 2024 · 7 comments · Fixed by #27
Closed

[Feature]: Add support for the SHA-256 auth algorithm #25

jwillikers opened this issue Jan 6, 2024 · 7 comments · Fixed by #27
Labels

Comments

@jwillikers
Copy link
Contributor

Describe the feature request

I'd like to use this on a host that uses SHA-256 for the auth algorithm, but it doesn't look like it's supported.

@jwillikers jwillikers added feature needs-triage Needs to be triaged labels Jan 6, 2024
@martialblog
Copy link
Member

@RincewindsHat can you have a look at this?

@martialblog
Copy link
Member

Maybe we could extend the auth-proto flag to include a SHA256 option?

@jwillikers Pull Requests are welcome, if you want to implement this

@jwillikers
Copy link
Contributor Author

Sweet, I'll try to find some time to add that. Should be pretty trivial since it is supported in the net-snmp library.

@martialblog martialblog removed the needs-triage Needs to be triaged label Jan 8, 2024
@RincewindsHat
Copy link
Member

Jup, could be done in a way which would not disturb the other options.
If net-snmp supports it, this should be relatively easy.
Testing would be annoying though.

@jwillikers
Copy link
Contributor Author

jwillikers commented Jan 13, 2024

Jup, could be done in a way which would not disturb the other options. If net-snmp supports it, this should be relatively easy. Testing would be annoying though.

@RincewindsHat I found your branch and that works for me, although I think I've ran into an unrelated bug which I opened an issue for: #26.

@RincewindsHat
Copy link
Member

Good to know, thanks :-)
Wanted to add the help for that option and stuff and didn't find the time since then.

@jwillikers
Copy link
Contributor Author

I went ahead and updated the documentation and help output for you in #27.

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

Successfully merging a pull request may close this issue.

3 participants