Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: puppetlabs/puppetlabs-firewall
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.0.2
Choose a base ref
...
head repository: puppetlabs/puppetlabs-firewall
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.0.3
Choose a head ref
  • 14 commits
  • 11 files changed
  • 5 contributors

Commits on Oct 27, 2011

  1. (#10324) Remove extraneous whitespace from iptables rule line in spec…

    … tests
    
    The extra spaces between port numbers were messing up the parse and
    putting elements in the wrong parts of the hash during rule_to_hash.
    
    This wasn't causing a test problem now, but was creating an issue for
    other work people were doing on the module.
    kbarber committed Oct 27, 2011
    Configuration menu
    Copy the full SHA
    83663b8 View commit details
    Browse the repository at this point in the history
  2. Merge pull request #26 from kbarber/ticket/10324-extra_spaces

    (#10324) Remove extraneous whitespace from iptables rule line in spec tes
    dcarley committed Oct 27, 2011
    Configuration menu
    Copy the full SHA
    2300fed View commit details
    Browse the repository at this point in the history

Commits on Oct 30, 2011

  1. (#9082) Sort iptables --state option values internally to keep it con…

    …sistent across runs
    
    Previously we were getting multiple re-runs due to the fact that iptables
    returns a different order with iptables-save then what was used when creating
    the rule.
    
    This patch fixes that by sorting states with should=.
    
    Added unit tests to ensure states are correctly sorted. Also added comments in
    code to ensure people understand why
    Chris Boulton authored and kbarber committed Oct 30, 2011
    Configuration menu
    Copy the full SHA
    fc3ecdc View commit details
    Browse the repository at this point in the history
  2. Merge branch 'ticket/9082-state-change'

    * ticket/9082-state-change:
      (#9082) Sort iptables --state option values internally to keep it consistent across runs
    kbarber committed Oct 30, 2011
    Configuration menu
    Copy the full SHA
    f1b73a9 View commit details
    Browse the repository at this point in the history

Commits on Nov 10, 2011

  1. Configuration menu
    Copy the full SHA
    4595f39 View commit details
    Browse the repository at this point in the history

Commits on Nov 12, 2011

  1. Merge branch 'ticket/10700-comment_validation'

    * ticket/10700-comment_validation:
      (#10700) allow additional characters in comment string
    kbarber committed Nov 12, 2011
    Configuration menu
    Copy the full SHA
    99e9e05 View commit details
    Browse the repository at this point in the history
  2. (#10690) Create new port property

    This new property will allow you to specify ports that match both destination
    and source.
    
    This works the same as dport and sport parameters, so it provides array
    support and hyphen separated ranges of ports as well.
    saysjonathan authored and kbarber committed Nov 12, 2011
    Configuration menu
    Copy the full SHA
    8040784 View commit details
    Browse the repository at this point in the history
  3. Merge branch 'ticket/10690-multiport_ports'

    * ticket/10690-multiport_ports:
      (#10690) Create new port property
    kbarber committed Nov 12, 2011
    Configuration menu
    Copy the full SHA
    e73cc9d View commit details
    Browse the repository at this point in the history
  4. (#10693) Ensure -m limit is added for iptables when using 'limit' param

    Previously we had only been adding --limit to the iptables arguments
    which meant the 'limit' parameter was not working at all. This patch
    fixes that.
    saysjonathan authored and kbarber committed Nov 12, 2011
    Configuration menu
    Copy the full SHA
    9c87087 View commit details
    Browse the repository at this point in the history
  5. Merge branch 'ticket/10693-limit_module'

    * ticket/10693-limit_module:
      (#10693) Ensure -m limit is added for iptables when using 'limit' param
    kbarber committed Nov 12, 2011
    Configuration menu
    Copy the full SHA
    0fb94da View commit details
    Browse the repository at this point in the history

Commits on Nov 13, 2011

  1. Configuration menu
    Copy the full SHA
    c5db2c2 View commit details
    Browse the repository at this point in the history
  2. Merge branch 'maint/fix_copyright_and_licensing'

    * maint/fix_copyright_and_licensing:
      (maint) Fix licensing references to GNU GPL.
    kbarber committed Nov 13, 2011
    Configuration menu
    Copy the full SHA
    a6d9810 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    71c54d5 View commit details
    Browse the repository at this point in the history
  4. Merge pull request #32 from kbarber/ticket/10792-release_0.0.3

    (#10792) Release 0.0.3 - CHANGELOG and Modulefile update
    ghoneycutt committed Nov 13, 2011
    Configuration menu
    Copy the full SHA
    05ef948 View commit details
    Browse the repository at this point in the history
Loading