Skip to content

Conversation

nehro-em
Copy link

@nehro-em nehro-em commented Jul 1, 2025

Adds support for marking an SNMP auth block as the default by specifying default: true in the YAML configuration. When present, the marked auth is assigned to DefaultAuth and used wherever a default is needed. If multiple auths are marked as default, an error is returned.

This makes the default behavior configurable without code changes.

nehro-em and others added 2 commits July 1, 2025 13:46
Adds support for marking an SNMP auth block as the default by specifying `default: true`
in the YAML configuration. When present, the marked auth is assigned to `DefaultAuth`
and used wherever a default is needed. If multiple auths are marked as default, an error is returned.

This makes the default behavior configurable without code changes.
@bastischubert
Copy link
Member

Hello @nehro-em ,

in order to get this PR merged you'd need to sign the commit - once thats done i'm happy to approve the changes and merge it .

Thx for your contribution!

@nehro-em
Copy link
Author

Merge to branch main

@bastischubert
Copy link
Member

Copy link
Author

@nehro-em nehro-em left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Signed-off-by: Nehro E.M [email protected]

@SuperQ
Copy link
Member

SuperQ commented Jul 31, 2025

This needs a DCO sign-off. You can use git commit -s --amend to add it.

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 this pull request may close these issues.

3 participants