Commit Graph

179 Commits (0e64863429575781b437e19f4065d729d1684373)

Author SHA1 Message Date
Brian Geihsler 0e64863429 Fix #1099: Only populate site result when it is there 9 years ago
Brian Coca 561d768f44 Merge pull request #1076 from trondhindenes/win_package_bugfix
turned off strict msi code checking
9 years ago
Brian Coca a155f7ee34 Merge pull request #1071 from nlandais/devel
Adding fix to get around bug found with the New-Website command when…
9 years ago
Brian Coca cc69ac1513 Merge pull request #1064 from dreamcat4/feature/win_fw-hotfixes
hotfixes: module windows/win_firewall_rules
9 years ago
Trond Hindenes b9d842ecd6 turned out strict msi code checking 9 years ago
Nicolas Landais d1dcf8e06b Adding fix to get around bug found with the New-Webstire command when running playbook on a vanilla machine. 9 years ago
= 576b5e2fc3 Document module limitations to resolve https://github.com/ansible/ansible-modules-extras/issues/908 9 years ago
Dreamcat4 ece9c2b43a fix: Add 'enable:' flag for enabling existing rules which are disabled by default.
This is a very much needed flag. To turn on/off existing firewall rules. And like the recent fix of the 'Profile' key, the netsh cmd prints 'Enabled' in the textual output. (at least on win10 it does). So again a similar small code added for the necessary exception handling when the difference check happens.

Please merge / push upstream like the other fixes. Many thanks. This is the last fix I have put together for this patch set. So I will raise my PR now.

But if you want to fix more bugs, it seems there may be others. In terms of the control code. Sometimes it will delete a rule under 'force' condition (when found difference) - but instead it is supposed to just modify the existing rule. Some weird behaviour regarding that. The other problem is that ansible does not return the error text printed by 'netsh' cmd verbatim... but it should as that makes debugging these errors a *lot* easier.
9 years ago
Dreamcat4 dcaa794949 fix: update documentation with new module name "win_firewall_rule" 9 years ago
Dreamcat4 469d22df97 fix: The names of firewall profiles are different on win10 & win2008r2
Hi again. This commit removes a small portion of your script's own internal error checking. In specific: for the value of the profile: key. This is essential to avoid errors on other verisons of the windows operating system which are not win2008r2 (your version).

For example: on win10 (and most likely win8x too), the names of the profiles don't include the values 'current' and 'all'. But instead the values are 'Public' 'Private' 'Domain' and 'Any. But in addition, there are also certain combinatorial values, such as profile=Public,Private etc. Which is too many to error check yourself.

Yet removing the error checking here should not cause any ill effects however: since the netsh advfirewall ... cmds themselves to add / remove / modify actually to their own error checking of the profile=value. So when the cmd is run, it will error out itself with an appropriate / informative error msg. No harm done.

Therefore please remove the highlighed portions from your own script. It is essential for interoperability with win10 and win8x. Many thanks.
9 years ago
Dreamcat4 6c5a4a14ef fix: win10 - Add exception handling for 'Profiles:' textual output key name mismatch.
In win10 (and pribably win8x also):

The output of 'show rule' key includes the line "Profiles:<TAB>Public,Private".
Yet your script expects the key name printed out to be "Profile:<TAB>value".

This commit added the necessary exception handling to avoid flagging 'different=true' under the false circumstance. The key name to SET a firewall rule is still "profile=" and not "profiles=".

There is coming up another commit to fix the value handling for win10/win8. Which is another (different) error with the profile: key.
9 years ago
Dreamcat4 2654789af7 fix: fw rule names must always be quoted, to permit spaces ' ' and brackets '()'
Without this fix, the 'netsh' command gets name=Firewall Rule Name instead of name="Firewall Rule Name". Thus causing all sorts of havoc. Basic shell quoting rules seems to apply to Windows Powershell too. This is very much needed as many of windows 10's default firewall rules contain spaces and brackets () characters.
9 years ago
Brian Coca ebbd7748b1 doc formatting fixes 9 years ago
Brian Coca 02b68be09d Merge pull request #854 from nitzmahone/win_updates_rewrite
win_updates rewrite for 2.0
9 years ago
Trond Hindenes 7c48fe617d added module author 9 years ago
Trond Hindenes 161b3cf1e1 fixes #984 9 years ago
Toshio Kuratomi 255666a09d Fix doc build 9 years ago
Greg DeKoenigsberg 22a3ec05e4 Merge pull request #1022 from trondhindenes/trondhindenes_win_chocolatey_improvements
win_chocolatey bugfixes
9 years ago
Greg DeKoenigsberg 5931e60d99 Merge pull request #695 from georgefrank/win_nssm
Adding win_nssm module
9 years ago
Greg DeKoenigsberg ff0a38ea65 Merge pull request #665 from perzizzle/scheduledTask
Ability to add/remove scheduled task
9 years ago
Trond Hindenes d7cc0f60c9 win_chocolatey bugfixes 9 years ago
Brian Coca 7f245e8c73 Merge pull request #318 from TimothyVandenbrande/win_fw
windows firewall control
9 years ago
TimothyVandenbrande 2d6303b368 upon request, added the license 9 years ago
Michael Perzel 123a2b25ed Remove duplicate action declaration. 9 years ago
Michael Perzel a7675e6621 Note parameters that are not idempotent 9 years ago
Michael Perzel 685c935a37 Documentation updates 9 years ago
Michael Perzel 64416ff094 Fix logging of error message 9 years ago
Michael Perzel de1696cb74 Add support for specifying user for scheduled task to run as 9 years ago
Michael Perzel 027dff6d3d Add support for command arguments 9 years ago
Michael Perzel d0b4bc0dda Show order of operations with parenthesis 9 years ago
Michael Perzel a9eb69b0fa Use helper methods to validate vars. Cleanup logic. 9 years ago
Toshio Kuratomi 6a3cf63351 correct documentation formatting 9 years ago
Brian Coca b0b0ac1a58 Merge pull request #713 from trondhindenes/win_package
Added module win_package (was #618)
9 years ago
nitzmahone 29a80d3555 win_updates rewrite for 2.0
uses scheduled job to run under a local token (required for WU client)
supports check mode
no external PS module deps
9 years ago
gfrank b75f7cc22b Move comments to the documentation 9 years ago
gfrank 9d68a1746f Remove unnecessary aliases arrays 9 years ago
gfrank ee395ced4d Clean up unneeded comment and comma 9 years ago
Chris Church 6a7b2f5869 Workaround for bug in PowerShell Get-Website cmdlet. 9 years ago
Brian Coca e30d8b84fe more doc fixes 9 years ago
Greg DeKoenigsberg e1b15e6930 Merge pull request #438 from petemounce/win_webpicmd
win_webpicmd module for IIS module installation
9 years ago
Greg DeKoenigsberg 34bd09aae1 Merge pull request #440 from petemounce/win_dotnet_ngen
Create win_dotnet_ngen module
9 years ago
Peter Mounce 8128d42dcd re-submit @adematte's PR #278 against HEAD 9 years ago
Michael Perzel 681cdd1c12 Remove accidental newlines 9 years ago
Michael Perzel cdb36ea0fc Cleanup enable/disable logic 9 years ago
Phil b598161f46 minor doc fixes that had lingering description of deprecated functions 9 years ago
Brian Coca cc305adfb6 minor doc fixes 9 years ago
Brian Coca 2754157d87 minor doc fixes 9 years ago
Trond Hindenes a2166b60cb Added module win_package 10 years ago
Greg DeKoenigsberg c6e069dead Merge pull request #174 from schwartzmx/module/win_unzip
win_unzip Module
10 years ago
gfrank 05d5d939e7 Adding nssm requirement note 10 years ago