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.

Sign up for GitHub

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

Jump to bottom

Outgoing port permit/avoid order wrong when outgoing_port_permit_first = false #313

Open
shieldwed opened this issue May 11, 2023 · 0 comments · May be fixed by #323
Open

Outgoing port permit/avoid order wrong when outgoing_port_permit_first = false #313

shieldwed opened this issue May 11, 2023 · 0 comments · May be fixed by #323

Comments

@shieldwed
Copy link

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 7.23.0
  • Ruby: 2.7.7p221
  • Distribution: Ubuntu 20.04.6 LTS (Focal Fossa)
  • Module version: 6.0.0

How to reproduce (e.g Puppet code you use)

  class { 'unbound':
    # ...
    outgoing_port_avoid        => '65535',
    outgoing_port_permit       => '65535',
    outgoing_port_permit_first => false,
    # ...
  }

What are you seeing

unbound.conf

  ...
  outgoing-port-permit: "65535"
  outgoing-port-avoid: "65535"
  ...

What behaviour did you expect instead

unbound.conf

  ...
  outgoing-port-avoid: "65535"
  outgoing-port-permit: "65535"
  ...

Output log

It should not be required

Any additional information you'd like to impart
b4ldr added a commit to b4ldr/puppet-unbound that referenced this issue Oct 24, 2023
@b4ldr
outgoing_port_permit_first: update to actually invert order
48b435d 
Currently this parameter has no effect.  This PR updates the code so the
order of the values is actually inverted

fixes voxpupuli#313
@b4ldr b4ldr linked a pull request Oct 24, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

outgoing_port_permit_first: update to actually invert order b4ldr/puppet-unbound
1 participant
@shieldwed