Results 254 comments of Daenney

So looking at #619 I'm a bit fuzzy on what would the solution would end up looking like. The Google doc states the issue about auth being stored in `snmp.yml`,...

That definitely helps, thanks! One thing I'm still wondering about, how do we make this reusable? Imagine a scenario where for whatever reason there's 2 sets of devices, both queryable...

Right, yes. I have no idea why this didn't click for me earlier. This makes sense. I might take a stab at a PR for this. On the format, you...

We seem to have the same problem with `WalkParams` in general. If we only extract `Auth`, then we're still keeping timeout, retries etc. inside a module. It feels like that...

I'm also wondering if we should stick `version` in `auth` or in `parameters`. The version dictates something about auth, but SNMPv3 also introduced other capabilities (even though we don't use...

I don't quite see how. From where I'm standing a module should define what to query, and potentially how to associate data with each other. Why would retries and timeouts...

Thinking more about this, it feels like retry and timeout should be query parameters on the target, instead of a named parameter group. So you'd end up with: `http://localhost:9116/snmp?module=if_mib&target=1.2.3.4&auth=foo&timeout=10&retry=5` instead...

You need to specify a file, `mibs/` is just a directory and GitHub doesn't allow directory listings. For example: https://raw.githubusercontent.com/net-snmp/net-snmp/v5.9/mibs/AGENTX-MIB.txt. You can find the listing on https://github.com/net-snmp/net-snmp/tree/v5.9/mibs. As you can...

I built the PR and I can confirm it works with my Naga X too. Thanks so much for taking my half-assed attempt and turning it into something properly working...

There's #1677 now which should hopefully work. I'm still waiting on the actual hardware to get delivered, so that PR is currently marked as draft since it's based purely on...