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

Check if raw string contains ipv4 address #171

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Check if raw string contains ipv4 address #171

wants to merge 3 commits into from

Conversation

foxdalas
Copy link

Description

[Describe what this change achieves]

Issues Resolved

[List any existing issues this PR resolves]

Check List

@martinb3 martinb3 added Priority: Medium Will bring visible benefit to the project Status: Pending Contributor Response labels May 30, 2017
@troyready
Copy link

Obviously some housecleaning is needed here (e.g. issue description, taking out the version bump), but it would be nice to something like this included.

Currently, if I specify a raw rule with an ipv4 address in it, it gets added to /etc/iptables/rules.v6 which causes the netfilter service restart to fail (more specifically /usr/share/netfilter-persistent/plugins.d/25-ip6tables start exits non-zero).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xorima xorima xorima approved these changes

Assignees
No one assigned
Labels
Priority: Medium Will bring visible benefit to the project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@foxdalas @troyready @xorima @martinb3 @tas50 @xorimabot