]>
review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/log
David Swan [Fri, 8 Jun 2018 10:01:43 +0000 (11:01 +0100)]
[FM-7044] Addition of Debian 9 support to firewall
Helen [Thu, 31 May 2018 08:49:56 +0000 (09:49 +0100)]
Merge pull request #764 from david22swan/FM-6961
[FM-6961] Removal of unsupported OS from firewall
David Swan [Wed, 30 May 2018 11:38:37 +0000 (12:38 +0100)]
[FM-6961] Removal of unsupported OS from firewall
david22swan [Fri, 18 May 2018 14:53:07 +0000 (15:53 +0100)]
Merge pull request #763 from puppetlabs/pdksync_heads/
master-0-g34e3266
pdksync - Update using 1.5.0
Helen Campbell [Fri, 18 May 2018 14:17:07 +0000 (15:17 +0100)]
david22swan [Fri, 18 May 2018 09:15:07 +0000 (10:15 +0100)]
Merge pull request #762 from puppetlabs/revert-761-pdksync_heads/
master-0-gfe44b98
Revert "pdksync - Update using 1.5.0"
Helen [Thu, 17 May 2018 15:54:55 +0000 (16:54 +0100)]
Revert "pdksync - Update using 1.5.0"
david22swan [Wed, 16 May 2018 15:16:36 +0000 (16:16 +0100)]
Merge pull request #761 from puppetlabs/pdksync_heads/
master-0-gfe44b98
pdksync - Update using 1.5.0
Helen Campbell [Wed, 16 May 2018 14:20:40 +0000 (15:20 +0100)]
david22swan [Fri, 11 May 2018 12:29:33 +0000 (13:29 +0100)]
Merge pull request #760 from pmcmaw/unmanage_gitlab
(MODULES-7153) - Unmanage gitlab-ci.yml
Paula McMaw [Fri, 11 May 2018 10:38:04 +0000 (11:38 +0100)]
(MODULES-7153) - Unmanage gitlab-ci.yml
david22swan [Fri, 27 Apr 2018 11:02:08 +0000 (12:02 +0100)]
Merge pull request #759 from pmcmaw/update_sync
(MODULES-7023) - Removing duplication in .sync.yml
Paula McMaw [Thu, 26 Apr 2018 13:46:55 +0000 (14:46 +0100)]
(MODULES-7023) - Removing duplication in .sync.yml
david22swan [Wed, 21 Feb 2018 12:00:21 +0000 (12:00 +0000)]
Merge pull request #749 from mikkergimenez/modules_2119_workaround_for_ip6tables_delete
(MODULES-2119) iptables delete -p all exception
david22swan [Wed, 21 Feb 2018 11:55:19 +0000 (11:55 +0000)]
Merge pull request #756 from mirekys/master
(MODULES-6129) negated option with address mask bugfix
Miroslav Bauer [Fri, 26 Jan 2018 21:59:39 +0000 (22:59 +0100)]
(MODULES-6129) negated option with address mask bugfix
willmeek [Thu, 25 Jan 2018 12:07:03 +0000 (12:07 +0000)]
Merge pull request #755 from puppetlabs/release
Release 1.12.0 merge back
Helen [Wed, 24 Jan 2018 14:04:08 +0000 (14:04 +0000)]
Merge pull request #754 from pmcmaw/1.12.0_release
Release prep 1.12.0
Paula McMaw [Wed, 24 Jan 2018 11:57:28 +0000 (11:57 +0000)]
Release prep 1.12.0
Paula Muir [Wed, 24 Jan 2018 11:47:43 +0000 (11:47 +0000)]
(MODULES-6455) - PDK convert firewall (#753)
* Rubocop and puppet-lint errors
* pdk convert firewall
* Remove bundler update
david22swan [Tue, 23 Jan 2018 17:28:12 +0000 (17:28 +0000)]
Merge pull request #752 from pmcmaw/convert
(maint) - Address rubocop & puppet lint errors
Paula McMaw [Tue, 23 Jan 2018 16:35:16 +0000 (16:35 +0000)]
Address rubocop and lint errors
Paula Muir [Mon, 8 Jan 2018 15:01:46 +0000 (15:01 +0000)]
Merge pull request #750 from puppetlabs/msync_MODULES-6339
(maint) modulesync
65530a4 Update Travis
Michael T Lombardi [Wed, 3 Jan 2018 22:11:41 +0000 (16:11 -0600)]
(maint) modulesync
65530a4 Update Travis
Related: https://github.com/puppetlabs/modulesync_configs/pull/177
Mikker Gimenez-Peterson [Thu, 21 Dec 2017 02:44:37 +0000 (18:44 -0800)]
(MODULES-2119) iptables delete -p all exception
This change works around a bug in ip6tables where rules will not be deleted if they attempt to match the 'all' protocol, as it does not properly handle a missing protocol field as an implicit 'all':
netfilter bug located here: https://bugzilla.netfilter.org/show_bug.cgi?id=1015
david22swan [Wed, 13 Dec 2017 15:19:09 +0000 (15:19 +0000)]
RubocopFix (#746)
david22swan [Wed, 13 Dec 2017 14:50:59 +0000 (14:50 +0000)]
Merge pull request #745 from puppetlabs/maint_modsync_384f4c1
(maint) - modulesync
384f4c1
tphoney [Wed, 13 Dec 2017 10:49:41 +0000 (10:49 +0000)]
(maint) - modulesync
384f4c1
willmeek [Tue, 12 Dec 2017 11:16:36 +0000 (11:16 +0000)]
Merge pull request #737 from mihall-primus/centos5
(MODULES-6092) Set correct seluser for CentOS/RHEL 5.x
Paula McMaw [Fri, 8 Dec 2017 12:21:34 +0000 (12:21 +0000)]
Merge pull request #744 from hantona/negation-with-dash-in-chain
MODULES-6261: Fix error parsing rules with dashes in the chain name
EC2 Default User [Thu, 7 Dec 2017 20:43:47 +0000 (20:43 +0000)]
Fix error parsing rules with dashes in the chain name
Paula McMaw [Thu, 7 Dec 2017 10:00:53 +0000 (10:00 +0000)]
Merge pull request #743 from david22swan/Rubocop
Addressing Rubocop Errors
David Swan [Wed, 6 Dec 2017 15:42:27 +0000 (15:42 +0000)]
FullFix
TP Honey [Wed, 6 Dec 2017 15:39:14 +0000 (15:39 +0000)]
Merge pull request #742 from david22swan/Rubocop
QuickFix
David Swan [Wed, 6 Dec 2017 15:37:53 +0000 (15:37 +0000)]
QuickFix
david22swan [Thu, 30 Nov 2017 10:02:12 +0000 (10:02 +0000)]
Merge pull request #741 from puppetlabs/release
Release mergeback 1.11.0
TP Honey [Wed, 29 Nov 2017 15:08:48 +0000 (15:08 +0000)]
Merge pull request #740 from david22swan/PreRelease
1.11.0PreRelease
David Swan [Wed, 29 Nov 2017 14:34:37 +0000 (14:34 +0000)]
1.11.0PreRelease
david22swan [Wed, 29 Nov 2017 10:25:11 +0000 (10:25 +0000)]
Rubocop Implemented (#735)
* Rubocop Implemented
Paula McMaw [Fri, 24 Nov 2017 12:56:51 +0000 (12:56 +0000)]
Merge pull request #739 from jistr/remove-debug-puts
Remove leftover debugging 'puts'
Jiri Stransky [Thu, 23 Nov 2017 15:37:07 +0000 (16:37 +0100)]
Remove leftover debugging 'puts'
I used the puts to figure out rspec behavior and forgot to remove it
before commiting a fix.
Paula McMaw [Thu, 23 Nov 2017 14:50:40 +0000 (14:50 +0000)]
Merge pull request #738 from jistr/skip-unparsable-iptables-rules
(MODULES-6029) Skip unparsable rules with warning
Jiri Stransky [Thu, 23 Nov 2017 14:11:03 +0000 (15:11 +0100)]
(MODULES-6029) Skip unparsable rules with warning
The iptables rules parser has very strict/simplistic expectations
about how iptables rules should look like, and can easily fail to
parse rules that weren't produced by the module itself.
We should ignore the unfitting rules when parsing and produce a
warning rather than causing a fatal error and stopping the Puppet run.
Michael Hall [Wed, 22 Nov 2017 15:37:30 +0000 (10:37 -0500)]
(MODULES-6092) Set correct seluser for CentOS/RHEL 5.x
In 1.10.0, applying the puppetlabs-firewall module on CentOS/RHEL 5.x produces warnings:
Warning: Failed to set SELinux context unconfined_u:object_r:etc_t on /etc/sysconfig/iptables
Notice: /Stage[main]/Firewall::Linux::Redhat/File[/etc/sysconfig/iptables]/seluser: seluser changed 'system_u' to 'unconfined_u'
Warning: Failed to set SELinux context unconfined_u:object_r:etc_t on /etc/sysconfig/ip6tables
Notice: /Stage[main]/Firewall::Linux::Redhat/File[/etc/sysconfig/ip6tables]/seluser: seluser changed 'system_u' to 'unconfined_u'
This commit adds a case to set the correct seluser.
david22swan [Fri, 17 Nov 2017 14:42:00 +0000 (14:42 +0000)]
Merge pull request #736 from puppetlabs/maint_modsync_1d81b6a
(maint) - modulesync
1d81b6a
Paula McMaw [Fri, 17 Nov 2017 12:16:30 +0000 (12:16 +0000)]
TP Honey [Tue, 14 Nov 2017 10:29:20 +0000 (10:29 +0000)]
Merge pull request #734 from puppetlabs/release
1.10.0 Mergeback
Paula McMaw [Fri, 10 Nov 2017 12:16:33 +0000 (12:16 +0000)]
Merge pull request #733 from jbondpdx/release
clean up firewall_multi section, minor fixes
jbondpdx [Thu, 9 Nov 2017 22:37:03 +0000 (14:37 -0800)]
clean up firewall_multi section, minor fixes
Paula McMaw [Wed, 8 Nov 2017 09:42:13 +0000 (09:42 +0000)]
Merge pull request #732 from david22swan/PreRelease
(FM-6577) PreRelease
David Swan [Tue, 7 Nov 2017 16:31:55 +0000 (16:31 +0000)]
PreReleaseFM-6577
Paula McMaw [Tue, 7 Nov 2017 14:15:06 +0000 (14:15 +0000)]
Merge pull request #731 from puppetlabs/release
1.9.0 Mergeback
david22swan [Tue, 7 Nov 2017 13:50:34 +0000 (13:50 +0000)]
Merge branch 'master' into release
david22swan [Tue, 7 Nov 2017 09:12:59 +0000 (09:12 +0000)]
Merge pull request #729 from sathlan/master
[MODULES-5924] Fix unmanaged rule regex when updating a iptable.
TP Honey [Mon, 6 Nov 2017 15:52:18 +0000 (15:52 +0000)]
Merge pull request #730 from david22swan/2017.3Fix
(QA)Fix to 2017.3 tests
David Swan [Mon, 6 Nov 2017 15:20:06 +0000 (15:20 +0000)]
(QA)Fix to 2017.3 tests
Sofer Athlan-Guyot [Fri, 3 Nov 2017 12:42:31 +0000 (13:42 +0100)]
[MODULES-5924] Fix unmanaged rule regex when updating a iptable.
We have two kind of unmanaged rules:
- empty comment: those get an md5 hash as a name prefixed with 90XX
- comment not matching “\d[graph,space]+$”
This is defined there[1]
But when we calculate the updated line we take into account only the
first kind of unmanaged rule in the offset[2] which leads to error.
Relaxing the regex solves this.
[1] https://github.com/puppetlabs/puppetlabs-firewall/blob/master/lib/puppet/provider/firewall/iptables.rb#L610..L617
[2] https://github.com/puppetlabs/puppetlabs-firewall/blob/master/lib/puppet/provider/firewall/iptables.rb#L802..L803
david22swan [Fri, 3 Nov 2017 09:27:54 +0000 (09:27 +0000)]
Merge pull request #666 from comel/parser-sanity-check-1
(MODULES-4200) Add simple sanity check for the rule to hash parser
david22swan [Thu, 2 Nov 2017 14:10:32 +0000 (14:10 +0000)]
Merge pull request #682 from crispygoth/add_gid_lookup
Add gid lookup
david22swan [Wed, 1 Nov 2017 12:23:31 +0000 (12:23 +0000)]
Merge pull request #728 from alexharv074/document_firewall_multi
(docs) Add missing hyperlink
Alex Harvey [Wed, 1 Nov 2017 07:37:24 +0000 (18:37 +1100)]
(docs) Add missing hyperlink
Left out in previous commit.
Eric Putnam [Tue, 31 Oct 2017 22:04:28 +0000 (15:04 -0700)]
Merge pull request #723 from alexharv074/document_firewall_multi
(docs) Add some notes about firewall_multi
Eric Putnam [Tue, 31 Oct 2017 20:37:37 +0000 (13:37 -0700)]
Merge pull request #727 from hunner/deb_9
(MODULES-5692) Match more than a single space
Hunter Haugen [Wed, 11 Oct 2017 20:30:18 +0000 (13:30 -0700)]
(MODULES-5692) Match more than a single space
Debian 9 outputs multiple spaces.
TP Honey [Tue, 3 Oct 2017 15:29:19 +0000 (16:29 +0100)]
Merge pull request #724 from HelenCampbell/revert722
Revert of PR 722 to undo the breaking changes introduced
Helen Campbell [Tue, 3 Oct 2017 14:11:17 +0000 (15:11 +0100)]
Revert of PR 722 to undo the breaking changes introduced
Alex Harvey [Fri, 29 Sep 2017 16:45:03 +0000 (02:45 +1000)]
(docs) Add some notes about firewall_multi
Clean up scattered references to this project so that docs make more
sense.
Eric Putnam [Wed, 27 Sep 2017 20:10:05 +0000 (13:10 -0700)]
Merge pull request #721 from kpengboy/resolve-ipv6
(MODULES-5645) Choose correct IP version for hostname resolution
Paula McMaw [Tue, 26 Sep 2017 12:54:58 +0000 (13:54 +0100)]
Merge pull request #722 from tphoney/stefanopini-patch-1
Correction of ICMPv6 protocol name
tphoney [Tue, 26 Sep 2017 11:38:42 +0000 (12:38 +0100)]
readme update
Stefano [Tue, 28 Feb 2017 09:36:56 +0000 (10:36 +0100)]
Correction of ICMPv6 protocol name
Correction of ICMPv6 protocol name from ipv6-icmp to icmpv6
TP Honey [Tue, 26 Sep 2017 11:21:06 +0000 (12:21 +0100)]
Merge pull request #694 from knackaron/ip6tables-disable
allow ip6tables to be disabled
Kevin Peng [Sun, 24 Sep 2017 07:43:42 +0000 (00:43 -0700)]
(MODULES-5645) Choose correct IP version for hostname resolution
Currently hostnames specified in a `source` or `destination` field
in a firewall rule are always resolved as IPv4, even when the
provider is `ip6tables`. Instead, intelligently determine whether
the hostname should be resolved as an IPv4 address or IPv6 address
based on the provider.
TP Honey [Fri, 22 Sep 2017 10:51:36 +0000 (11:51 +0100)]
Merge pull request #720 from puppetlabs/msync_18sep17_892c4cf
(maint) modulesync
892c4cf
Hailee Kenney [Mon, 18 Sep 2017 21:55:42 +0000 (14:55 -0700)]
(maint) modulesync
892c4cf
Paula McMaw [Fri, 8 Sep 2017 11:33:35 +0000 (12:33 +0100)]
Merge pull request #717 from tphoney/dummy_pr
remove whitespace at the EOL
Paula McMaw [Wed, 6 Sep 2017 13:25:06 +0000 (14:25 +0100)]
Merge pull request #719 from HelenCampbell/hashlimitfix
Update for hashlimit tests to not run on RedHat5 or Scientific 5
Helen Campbell [Wed, 6 Sep 2017 12:37:42 +0000 (13:37 +0100)]
Update for hashlimit tests to not run on RedHat5 or Scientific 5
Hashlimit doesn't seem to be fully supported on these OS'es so I've put a check to skip the tests.
TP Honey [Tue, 5 Sep 2017 14:13:30 +0000 (15:13 +0100)]
Merge pull request #718 from HelenCampbell/hashlimitfix
Yet another fix to hashlimit tests, smaller tests replace one large r…
Helen Campbell [Tue, 5 Sep 2017 13:27:34 +0000 (14:27 +0100)]
Yet another fix to hashlimit tests, smaller tests replace one large regex for match improvements.
TP Honey [Tue, 5 Sep 2017 09:58:26 +0000 (10:58 +0100)]
Merge pull request #716 from HelenCampbell/hashlimitfix
Fix to update hashlimit above test
Helen Campbell [Tue, 5 Sep 2017 08:50:28 +0000 (09:50 +0100)]
Unit test fix for travis
tphoney [Mon, 4 Sep 2017 14:48:05 +0000 (15:48 +0100)]
remove whitespace at the EOL
Helen Campbell [Mon, 4 Sep 2017 14:15:01 +0000 (15:15 +0100)]
Fix to update hashlimit above test
Helen [Tue, 29 Aug 2017 13:25:49 +0000 (14:25 +0100)]
Merge pull request #708 from ATIX-AG/master
MODULE-1805 Add hashlimit-module
TP Honey [Thu, 17 Aug 2017 14:34:00 +0000 (15:34 +0100)]
Merge pull request #715 from pmcmaw/rm_ubuntu
(MODULES-5501) - Remove unsupported Ubuntu
Paula McMaw [Thu, 17 Aug 2017 12:59:57 +0000 (13:59 +0100)]
(MODULES-5501) - Remove unsupported Ubuntu
Removing older version of Ubuntu that are not supported by the module.
Eric Putnam [Wed, 16 Aug 2017 21:54:15 +0000 (14:54 -0700)]
Merge pull request #713 from nbarrientos/sets
(MODULES-5340) Understand negated match sets
Nacho Barrientos [Thu, 27 Jul 2017 08:55:33 +0000 (10:55 +0200)]
(MODULES-5340) Understand negated match sets
This patch allows the provider to understand rules that contain negated
match sets when parsing the ruleset.
Paula McMaw [Fri, 28 Jul 2017 10:00:40 +0000 (11:00 +0100)]
Merge pull request #714 from puppetlabs/maint-modsync-
915cde70e20
(maint) modulesync
915cde70e20
Glenn Sarti [Thu, 27 Jul 2017 21:56:16 +0000 (14:56 -0700)]
(maint) modulesync
915cde70e20
Hunter Haugen [Mon, 10 Jul 2017 21:46:41 +0000 (14:46 -0700)]
Merge pull request #712 from puppetlabs/msync_modules5187
(MODULES-5187) mysnc puppet 5 and ruby 2.4
Eric Putnam [Thu, 6 Jul 2017 21:41:16 +0000 (14:41 -0700)]
(MODULES-5187) mysnc puppet 5 and ruby 2.4
Paula McMaw [Thu, 6 Jul 2017 08:59:43 +0000 (09:59 +0100)]
Merge pull request #711 from tphoney/MODULES-5177
(MODULES-5177) fixing tests for certain oses
tphoney [Wed, 5 Jul 2017 13:07:22 +0000 (14:07 +0100)]
(MODULES-5177) fixing tests for certain oses
Helen [Tue, 4 Jul 2017 15:43:50 +0000 (16:43 +0100)]
Merge pull request #710 from tphoney/MODULES-5173
(MODULES-5173) fix regex and test on el6
tphoney [Tue, 4 Jul 2017 14:13:06 +0000 (15:13 +0100)]
(MODULES-5173) fix regex and test on el6
TP Honey [Mon, 3 Jul 2017 11:31:53 +0000 (12:31 +0100)]
Merge pull request #709 from hunner/bump_metadata
(MODULES-5144) Prep for puppet 5
TP Honey [Mon, 3 Jul 2017 11:31:18 +0000 (12:31 +0100)]
Merge pull request #696 from traylenator/style
Puppet Lint Fixes
Hunter Haugen [Mon, 26 Jun 2017 21:04:54 +0000 (14:04 -0700)]
(MODULES-5144) Prep for puppet 5