search

jonnystorm / snmp-elixir

An SNMP client library for Elixir
Mozilla Public License 2.0
33 stars 12 forks source link

API Refactor #42

Closed jonnystorm closed 4 years ago

jonnystorm commented 5 years ago

I realize the current API is a bit broken in light of SNMP set (see #41). See the latest NetSNMP API for my ideas about how to fix it. If we're going to make breaking changes, it may be worth breaking things in that direction.

As for the credential API, this should probably be cut down to accepting a simple map. Maintaining two different ordinal input methods is a bit silly, even for me. Recent efforts to push credential generation/translation into its own module (see #40) are a good indicator that the excessive amount of code here is more of a distraction than it should be.

fbettag commented 4 years ago

Any updates on this? :) i would really like to switch over from net-snmp-elixir.

jonnystorm commented 4 years ago

@fbettag Thanks for your interest! #42 has been merged, and I'll be pushing 0.3.1 to Hex shortly.

fbettag commented 4 years ago

Thank you so much! 👌👌👌👌