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.

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

subnet 32 should not be stripped in ipset_sync #65

Closed
maxadamo opened this issue May 10, 2022 · 1 comment · Fixed by #66
Closed

subnet 32 should not be stripped in ipset_sync #65

maxadamo opened this issue May 10, 2022 · 1 comment · Fixed by #66
Labels
bug Something isn't working

Comments

@maxadamo
Copy link
Contributor

maxadamo commented May 10, 2022

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: n/a
  • Ruby: n/a
  • Distribution: n/a
  • Module version: 2.1.0

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

use an IPv6 network with 32 bits subnet

What are you seeing

/32 is being stripped from the script ipset_sync even on IPv6 networks

What behaviour did you expect instead

/32 should not be stripped on IPv6 because it's a legit subnet for IPv6 network

Any additional information you'd like to impart

this is the line in the script where the subnet is being stripped:
https://github.com/voxpupuli/puppet-ipset/blob/master/files/ipset_sync#L77

we are preparing a Pull Request

@maxadamo maxadamo changed the title slash 32 should not be stripped in ipset_sync subnet 32 should not be stripped in ipset_sync May 10, 2022
@maxadamo
Copy link
Contributor Author

duplicate of #30

@jhoblitt jhoblitt added the bug Something isn't working label Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants