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

How do you remove an existing Windows Firewall rule? #116

Open
mike406 opened this issue Apr 23, 2021 · 2 comments
Open

How do you remove an existing Windows Firewall rule? #116

mike406 opened this issue Apr 23, 2021 · 2 comments
Labels
docs Improvements or additions to documentation question

Comments

@mike406
Copy link

mike406 commented Apr 23, 2021

    windows_firewall::exception { 'OpenSSH-Server-In-TCP':
        ensure          => 'absent',
        direction       => 'in',
        action          => 'allow',
        display_name    => 'OpenSSH SSH Server (sshd)'
    }
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Left match operand must result in a String value. Got an Undef Value. (file: /etc/puppetlabs/code/environments/production/modules/windows_firewall/manifests/exception.pp, line: 106, column: 8) (file: /etc/puppetlabs/code/environments/production/modules/win_sshd/manifests/init.pp, line: 36)
@mike406
Copy link
Author

mike406 commented Apr 23, 2021

Figured it out...just need protocol and display_name

    windows_firewall::exception { 'OpenSSH-Server-In-TCP':
        ensure          => 'absent',
        protocol        => 'TCP',
        display_name    => 'OpenSSH SSH Server (sshd)'
    }

This module's readme (or error message) needs to have more clear indication of what parameters are required to get things done. Leaving this open so that can be addressed.

On another note, it would also be nice for this to support rules with protocol other than TCP, UDP, ICMPv4/v6

@kenyon kenyon added docs Improvements or additions to documentation question labels Apr 26, 2021
@ghoneycutt
Copy link
Member

@mike406 Could you please add this to the README?

https://github.com/voxpupuli/puppet-windows_firewall/edit/master/README.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation question
Projects
None yet
Development

No branches or pull requests

3 participants