net-mgmt/nrpe: allow hostnames in allowed_hosts#5305
Open
bkmorale wants to merge 1 commit intoopnsense:masterfrom
Open
net-mgmt/nrpe: allow hostnames in allowed_hosts#5305bkmorale wants to merge 1 commit intoopnsense:masterfrom
bkmorale wants to merge 1 commit intoopnsense:masterfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Important notices
Before you submit a pull request, we ask you kindly to acknowledge the following:
If AI was used, please disclose:
Related issue
If this pull request relates to an issue, link it here:
#4932
Describe the problem
I have noticed that when using NRPE in opnsense I am unable to add an allowed host that is a DNS entry and provides the error "Please specify a valid network segment or IP address." the current workaround I have is to manually update the nrpe file and add the DNS in the config file. the plugin will then work as expected but will break should anyone make changes to the NRPE config in the opnsense UI.
I have done this with existing servers in production and with newly deployed systems along with lab systems, I can confirm this is applicable to anything on version OPNsense 25.7.0 and above
Describe the proposed solution
Updated the General.xml to allow hostname entries in the allowed_host field