Merge pull request #1123 from tobias-urdin/fix-selinux-context

Fix SELinux context on newer CentOS

Merge pull request #1104 from cruelsmith/force-chain-delete

Force firewall chain delete

Merge pull request #1140 from david22swan/CONT-242/main/duplicate_rule

(CONT-242) Fix duplicate rule detection

Syntax Updates

Syntax updates

(CONT-242) Fix duplicate rule detection

This was previously accomplished b retrieving the full list of rules each time a rule was set in order to check it was unique. This was to allow the user to choose the response to a duplicate rule being found.

However this caused a massive slowdown within certain module runs and as such we have changed the the location of the check, so that it instead runs a check for any duplicates when retrieving the current rules prior to any updates being made.

As an effect of this the user is now unable to choose the response to a duplicate rule being found, however wee feel that this is a fair tradeoff for the increased speed and that the response that we have chosen is the correct one.

Merge pull request #1135 from puppetlabs/pdksync_CONT-1028-update-stdlib-dependency

pdksync - (MAINT) - Allow Stdlib 9.x

Bump minimum required stdlib version

Namespacing was introduced in Stdlib v9.0.0 and onwards, therefore any module affected by it need a minimum bump.

Co-authored-by: Tim Meusel <tim@bastelfreak.de>

Address namespaced functions

(CONT-1028) - Update Stdlib upper limit dependency

Merge pull request #1130 from puppetlabs/pdksync_CONT-1001-update_litmus_dep

pdksync - (CONT-1001) Add litmus ~> 1.0

(CONT-1001) Add litmus ~> 1.0

Merge pull request #1129 from puppetlabs/pdksync_CONT-930-rollout_new_gh-changelog

pdksync - (CONT-930) - Roll out new changelog generator

(CONT-930) - Roll out new changelog generator

Fix SELinux context on newer CentOS

Fix the SELinux contexts for CentOS Stream 8
and CentOS Stream 9.

Merge pull request #1122 from puppetlabs/pdksync_CONT-844-update_.sync.yml

pdksync - (CONT-844) - Update .sync.yml

(CONT-844) - Update .sync.yml

Merge pull request #1121 from puppetlabs/release-prep

Release prep v5.0.0

Release prep v5.0.0

Merge pull request #1118 from david22swan/CONT-779/puppet_8_clean

(Cont 779) Add Support for Puppet 8 / Drop Support for Puppet 6

Merge pull request #1120 from puppetlabs/release-prep

Release prep v4.1.0

Release prep v4.1.0

(MAINT) PDK Update

(CONT-799) Update Rubocop Versions

New Versions:
rubocop = 1.48.1
rubocop-performance = 1.16.0
rubocop-rspec = 2.19.0

Also includes the reactivation of all previously disabled Cops

(CONT-799) Unpin puppet_litmus

(CONT-779) Unpin puppet_agent

(CONT-779) Move Puppet version support

Merge pull request #1117 from puppetlabs/pdksync_CONT-826/main/pin_puppet_litmus

pdksync - CONT-826/main/pin_puppet_litmus

(CONT-826) Temporarily pin puppet_litmus to 0.34.6 or less

Merge pull request #1115 from puppetlabs/CONT-751-Implement_reusable_workflows

(CONT-752) - Implement reusable workflows

(CONT-752) - Implement reusable workflows

Merge pull request #1113 from puppetlabs/maint-pin_puppet_agent_fixture

(MAINT) Pin puppetlabs-puppet_agent

(MAINT) Pin puppetlabs-puppet_agent

This change pins the puppetlabs-puppet_agent module to v4.12.1.

Previosuly, the fixutre was configured to pull from main. Given the
recent changes when moving towards puppet8, main is unsafe.

Merge pull request #1107 from buzzdeee/main

Ignore OpenBSD, similarly to FreeBSD

Merge pull request #1110 from puppetlabs/CONT-352-Syntax_update

(CONT-352) Syntax update

bugfix

parameter datatypes

top scope facts

relative classname inclusion

(CONT-352) Syntax update

Code now compliant with rules regarding:

legacy facts

Merge pull request #1108 from puppetlabs/pdksync_CONT-211-add_mend_config

pdksync - (CONT-211) Adding mend config

Adding mend file

Merge pull request #1095 from ahrefs/main

debian: service: fix `ensure` parameter usage

Ignore OpenBSD, similarly to FreeBSD

Merge pull request #1106 from puppetlabs/pdksync_CONT-494/pin_github_changelog_generator

pdksync - (CONT-494) Pin github_changelog_generator and JSON gem versions

(CONT-494) Fix for .sync.yml

(CONT-494) PDK Update

(CONT-494) Pin github_changelog_generator gem version

Merge pull request #1103 from robertc99/main

redhat9 needs iptables service

firewall: tests: add required `facts` declarations

https://github.com/puppetlabs/puppetlabs-firewall/actions/runs/3857494896/jobs/6575029953#step:9:1110

firewall: tests: add `debian` tests for `ensure => stopped`

also `with Debian unstable, ensure => stopped`, seems to have been overlooked

firewall: tests: update `contain_service` to `running` for `debian`

bc its necessary with the new changes https://github.com/puppetlabs/puppetlabs-firewall/pull/1095#issuecomment-1339654976

Merge branch 'puppetlabs:main' into main

Force firewall chain delete

Flush chain to get rid of unmanaged firewall rules in that chain
else the remove of the chain will fail.

redhat9 needs iptables service

Merge pull request #1101 from puppetlabs/pdksync_maint-remove_stale_bot

pdksync - (MAINT) Remove stalebot

(MAINT) Remove stalebot workflow

Merge pull request #1099 from puppetlabs/release-prep

Release prep v4.0.1

Release prep v4.0.1

(GH-1097) Bumping back required puppet version (#1098)

Prior to this commit, due to a mistake, there was a reversion to the
previously implemented bump to minimum required puppet version, which
was part of the CVE update.

This commit aims to address issue 1097 in Github which brings to light
the need for this puppet bump to be restored to 6.24

Merge pull request #1094 from puppetlabs/1093-fix_unresolved_fact_error

(1093) - Fix unresolved fact error

(1093) - Fix unresolved fact error

Prior to this commit, work was carried out on this module to update all instances of the now deprecated Facter::Util::Resolution, and replace all with its newer and supported counterpart Facter::Core::Execution.
However, these do not behave exactly the same. Facter::Util::Resolution initially ran a which to locate the binary before executing, preventing any errors from occuring. The newer Facter::Core::Execution method did not do this, instead it attempted to execut>

This commit aims to introduce an "on_fail:false" flag to each execute statement, so that a failed execute will return false (boolean) as oppose to an error, which can then be used for further logic.

Merge pull request #1085 from kjetilho/fix/iptables-el9

package "iptables" has been replaced by "iptables-nft" on EL9

package "iptables" has been replaced by "iptables-nft" on EL9

There are some pointers given by dnf about "iptables", but these confuse
Puppet into aborting with the error message:

```console
Error: /Stage[main]/Firewall::Linux/Package[iptables]: Could not evaluate: no implicit conversion of Array into Hash
```

Fedora had a similar patch in commit 486e4b5779f5069e which I think
fixed the bug https://tickets.puppetlabs.com/browse/MODULES-11147 but
the same issue rared its head here on AlmaLinux 9.0.

The RPM for iptables-legacy states:

> This package contains the legacy tools which are obsoleted by
> nft-variants in iptables-nft package for backwards compatibility reasons.
> If you need to set up firewalls and/or IP masquerading, you should not install
> this package but either nftables or iptables-nft instead.

Merge pull request #1096 from kjetilho/fix/nflog_size

support --nflog-size as replacement for --nflog-range

--nflog-range was deprecated way back in 2016.

https://git.netfilter.org/iptables/commit/?h=v1.6.1&id=7070b1f3c88a0c3d4e315c00cca61f05b0fbc882

debian: service: fix `ensure` parameter usage

`ensure` is defined but not used anywhere, no clear way to ensure the service is running
(and makes it impossible to do so outside the module)

Merge pull request #1092 from puppetlabs/release-prep

Release prep v4.0.0

Release prep v4.0.0

Merge pull request #1059 from cmusik/main

add support for using rpfilter in rules

add support for using multiple rpfilter options in rules

Merge pull request #1090 from puppetlabs/revert-1088-CONT-5-raise_minimum_puppet_required

Revert "(CONT-5) Raising minimum required puppet version"

Revert "(CONT-5) Raising minimum required puppet version"

Merge pull request #1087 from puppetlabs/CONT-5-Codebase_hardening

(CONT-5) Codebase hardening

Merge pull request #1088 from puppetlabs/CONT-5-raise_minimum_puppet_required

(CONT-5) Raising minimum required puppet version

(CONT-5) Raising minimum required puppet version

Prior to this commit, and as part of the codebase hardening project, we
sanitised some commands in the module. However, this sanitisation
method requires that users have at least puppet version 6.24.0 and
onwards to be effective.

This commit aims to resolve that requirement by raising the required
puppet version in the metadata.json file.

(CONT-5) Codebase hardening

Prior to this commit, and after the work perform in CONT-256, there were
a few commands in the module that were not properly sanitised according
to our current standards.

This PR aims to implement some changes that ensure no malformed commands
are passed through to the system. Primarily, the commands targeted were
the ones related to Open3 and exec.

Merge pull request #1084 from puppetlabs/CONT-256-module_cleanup

(CONT-256) Removing outdated code

Addressing comments

Rubocop corrections

Remove/Adjust Spec test cases

(CONT-256) Removing outdated code
Prior to this commit, the firewall modules had multiple instances of
outdated code.

This commit aims to clean-up some of that code to keep the module clean
and readable.

Merge pull request #1079 from puppetlabs/cont-173/main/updating_deprecated_facter

(CONT-173) - Updating deprecated facter instances

(CONT-173) - Updating deprecated facter instances

Prior to this PR, this module contained instances of Facter::Util::Resolution.exec and Facter::Util::Resolution.which, which are deprecated.
This PR aims to replace these exec helpers with their supported Facter::Core::Execution counterparts.

This PR:

Replaced all Facter::Util::Resolution instances with corresponding Facter::Core::Execution exec helpers

Merge pull request #1081 from puppetlabs/revert-1066-match-mark-negate

Revert "Add negate to match_mark"

fix lint

Revert "Add negate to match_mark"

Merge pull request #1066 from Enapter/match-mark-negate

Add negate to match_mark

Merge pull request #1078 from puppetlabs/pdksync_CONT-189/remove_os_support

pdksync - (CONT-189) Remove support for RedHat6 / OracleLinux6 / Scientific6

(CONT-189) Remove support for Scientific6

(CONT-189) Remove support for OracleLinux6

(CONT-189) Remove support for RedHat6

Merge pull request #1077 from puppetlabs/pdksync_pdksync/remove_puppet_module_gems

pdksync - (PDKSync) Removal of puppet_module_gems

Merge pull request #1075 from puppetlabs/pdksync_remove_debian_9

pdksync - (CONT-130) - Dropping Support for Debian 9

(PDKSync) Removal of puppet_module_gems

(CONT-130) Dropping Debian 9 Support

Merge pull request #1023 from kjetilho/fix_string_to_port

fix service port number lookup to use protocol

fix spurious fail, http/udp does not exist in test harness

Switch to "talk" (port 517), since that is an UDP only service in the default Debian and RedHat /etc/services.

fix service port number lookup to use protocol

The existing code passes `:proto`, which `string_to_port` casts to a
string, gets "proto", compares that to the possibilities "udp" or "tcp",
and when neither, falls back to using "tcp".

This patch passes the actual proto value to the function, in case there is
a UDP specific service in your /etc/services (uncommon, but it happens).
It looks like Puppet will evaluate the properties in declared order,
so I had to move `newproperty(:proto)` up so `@resource[:proto]` was
available in the code for `sport`, `dport` and `port`.

Merge pull request #1074 from puppetlabs/release_prep_v3.6.0

Release prep v3.6.0