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

Update main.yml #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update main.yml #11

wants to merge 1 commit into from

Conversation

flybyray
Copy link

port range support for firewalld could be done like this.

Might be helpful in #6

      firewall_services:
        - name: 30000-32767
          protocol: tcp
          range: true

name: Pull request
about: Describe the proposed change


Describe the change
A clear and concise description of what the pull request is.

Testing
In case a feature was added, how were tests performed?

port range support for firewalld could be done like this.

Might be helpful in [robertdebock#6](robertdebock#6)

```yaml
      firewall_services:
        - name: 30000-32767
          protocol: tcp
          range: true

```
@Pandemonium1986
Copy link

Hello @robertdebock and @flybyray .
I am interested in this feature.
@robertdebockt what would you expect in terms of testing in order to integrate this feature into a release. I am ready to contribute

@robertdebock
Copy link
Owner

Hi,

Please describe an example of how to use this (commented) in defaults/main.yml.

@flybyray
Copy link
Author

flybyray commented Mar 1, 2024

Actually I replaced firewall usage with the role linux-system-roles.firewall

But there are quite a lot of examples: https://linux-system-roles.github.io/firewall/#port

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

Successfully merging this pull request may close these issues.

3 participants