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

Get snmp context from param #1040

Closed
bastischubert opened this issue Nov 7, 2023 · 1 comment · Fixed by #1163
Closed

Get snmp context from param #1040

bastischubert opened this issue Nov 7, 2023 · 1 comment · Fixed by #1163

Comments

@bastischubert
Copy link
Member

SNMP Context exists with snmpv3 and is often used to group various management informations together. (details can be found here: https://datatracker.ietf.org/doc/html/rfc5343#section-2)

Currently we're able to use snmp contexts as part of the auths definition in snmp.yml which makes it quite hard to use them, especially whe you use one snmp context per vrf (for example). As the context is not security sensitive, we should allow to pass in the context via a url param
for example: /snmp?target=10.11.12.13&auth=mystrongauth&module=snmp_inventory&snmp_context=vrf-mgmt

@SuperQ
Copy link
Member

SuperQ commented Nov 8, 2023

I think we talked about this in a previous issue. (of course I can't find that right now) I think you're right about contextName being not sensitive. I'm slightly confused by the RFC since it's talking about a requirement for a contextEngineID as part of the request. The contextEngineID is considered security sensitive.

If the param is not sensitive, it should be named context_name to match the config file format.

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

Successfully merging a pull request may close this issue.

2 participants