puppet-windows_firewall icon indicating copy to clipboard operation
puppet-windows_firewall copied to clipboard

Missing 'state' check of existing firewall rule.

Open skasai opened this issue 9 years ago • 1 comments

In looking at the code, I noticed there is also an addition issue that may have not been considered in the windows_firewall that may need to be noted.

There is checks for the rule's existence, but not check if rule not only exists, but if parameters are the same or updated.

This means that the module will not update the rule if there is a change in the parameters.

IE: Port change or IP change.

This may have to be noted that a module will be needed created to remove old settings and applied before pushing new changes.

skasai avatar May 06 '15 19:05 skasai