GitHub puppet-windows_firewall
14
14
12
puppet module for configuring the windows firewall

Metadata Valid
No translation
Correct Puppet Version Range
Supported Puppet version range is %{PUPPET_VERSION_RANGE}
With Puppet Version Range
Puppet version range is present in requirements in metadata.json
With Operatingsystem Support
No translation
In Modulesync Repo
No translation
In Plumbing
Is in plumbing
Has Secrets
Has a .sync.yml file
Synced
Has a .msync.yml file
Latest Modulesync
No translation
Has Modulesync
Is present in voxpupuli/modulesync_config/managed_modules.yml
Released
Is in modulesync_config and in forge releases.
Reference Dot Md
The repository has a REFERENCE.md. It needs to be generated / puppet-strings documentation is missing.

Open Pull Requests

Implemented profile selection
merge-conflicts
tests-fail

<!--
Thank you for contributing to this project!

-->

Pull Request (PR) description

<!--
This PR adds an optional firewall exception parameter that specifies the network profiles, such as domain, private and/or public.
-->

This Pull Request (PR) fixes the following issues

<!--
n/a
-->

Port range
merge-conflicts

Pull Request (PR) description

This PR

  1. Fixes some typos
  2. Adds support for comma-separated port list and dash separated range for Local and Remote ports
  3. Adds support for 'RPC' and 'RPC-EPMap' port ranges for local ports
  4. Adds default description
  5. Makes sure ICMPv4/v6 protocol does not accept ports
  6. Makes sure 'RPC' and 'RPC-EPMap' only use TCP
  7. Adds Win2k3/XP specific check for rule existence

This Pull Request (PR) fixes the following issues

80

Fix truncated verification

<!--
Thank you for contributing to this project!

-->

Pull Request (PR) description

I ran in to an issue where '%SystemRoot%\System32\logfiles\firewall\publicfw.log' was not being validated properly. The powershell output of Get-NetFirewallProfile was getting wrapped so only '%SystemRoot%\System32\logfiles\firewall\publi' was on the first line. This resulted in the log file being changed every time Puppet ran. This change sets the output length to 4096, which should be more than enough for long file names. After changing it locally, I was able to run Puppet with the current setting properly being validated and again with it being told to make a change.

This Pull Request (PR) fixes the following issues

None that I can find.

Local and Remote port range and csv #80

Pull Request (PR) description

Fix the ability to use a range of integers as well as comma separated integers and update the README to reflect those changes. Add test for those port changes. Add a type alias and test for that alias.

This Pull Request (PR) fixes the following issues

Replace this comment with the list of issues or n/a.

Fixes #80