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

[FEATURE] Add priority to rich rules #51

Closed
Sxderp opened this issue Jun 2, 2021 · 1 comment · Fixed by #52
Closed

[FEATURE] Add priority to rich rules #51

Sxderp opened this issue Jun 2, 2021 · 1 comment · Fixed by #52

Comments

@Sxderp
Copy link

Sxderp commented Jun 2, 2021

Is your feature request related to a problem?

Newer versions of firewalld support a priority attribute for rich rules. Migrating from iptables to nftables I need this functionality as direct-rules can no longer be used for what I want.

Describe the solution you'd like

Add a rich_rule priority key such that <rule priority={{ rule.priority }}> is the output.

Additional context

I might get around to creating a PR. /shrug

Sxderp added a commit to Sxderp/firewalld-formula that referenced this issue Jun 18, 2021
Sxderp added a commit to Sxderp/firewalld-formula that referenced this issue Jun 18, 2021
Sxderp added a commit to Sxderp/firewalld-formula that referenced this issue Jun 18, 2021
@Sxderp Sxderp mentioned this issue Jun 18, 2021
19 tasks
Sxderp added a commit to Sxderp/firewalld-formula that referenced this issue Jun 18, 2021
Sxderp added a commit to Sxderp/firewalld-formula that referenced this issue Jun 18, 2021
Sxderp added a commit to Sxderp/firewalld-formula that referenced this issue Jun 18, 2021
@myii myii closed this as completed in #52 Jun 18, 2021
saltstack-formulas-travis pushed a commit that referenced this issue Jun 18, 2021
# [1.3.0](v1.2.1...v1.3.0) (2021-06-18)

### Continuous Integration

* add `arch-master` to matrix and update `.travis.yml` [skip ci] ([cdaade3](cdaade3))

### Features

* **rich-rules:** add priority to rich rules ([9c2b41d](9c2b41d)), closes [#51](#51)
@saltstack-formulas-travis

🎉 This issue has been resolved in version 1.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants