Fix minimum version requirement for stdlib

Since c0b8640fc6a7c3381877863960e02da3fa1fe640, we use the namespaced
version of `stdlib::ensure_packages`, which appeared in version 9.0.0 of
the stdlib module.

Adjust the version requirement accordingly.

Merge pull request #1113 from bastelfreak/pdksync_CONT-1028-update-stdlib-dependency

(CONT-1028) puppetlabs/stdlib: Allow 9.x

ensure_packages: switch to stdlib::ensure_packages

Switch from merge() to native puppet

(CONT-1028) puppetlabs/stdlib: Allow 9.x

Merge pull request #1110 from puppetlabs/CONT-581

(CONT-581)  Adding deferred function support for password field

(CONT-581)  Adding deferred function support for password field

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

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

(CONT-1001) Add litmus ~> 1.0

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

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

(CONT-930) - Roll out new changelog generator

Merge pull request #1104 from kenyon/refresh-on-expired-subkeys

(MODULES-10831) key is expired if all subkeys are expired

(MODULES-10831) key is expired if all subkeys are expired

Previously, subkeys were not considered at all in the determination of
whether a key was expired. Now this looks at all of the subkeys, and if
they are all expired, considers the whole key expired.

Merge pull request #1101 from david22swan/CONT-773/main/puppet_8

(CONT-773) Add Support for Puppet 8 / Remove Support for Puppet 6

(CONT-773) Lint/ConstantDefinitionInBlock Correction

(CONT-773) Rubocop Manual Fix 6 - RSpec/RepeatedExampleGroupDescription

(CONT-773) Rubocop Manual Fix 5 - RSpec/ContextWording

(CONT-773) Rubocop Manual Fix 4 - Performance/CollectionLiteralInLoop

(CONT-773) Rubocop Manual Fix 3 - Lint/ConstantDefinitionInBlock

(CONT-773) Rubocop Unsafe Auto Fixes 4-5

- Style/GlobalStdStream
- Style/SlicingWithRange

(CONT-773) Rubocop Unsafe Auto Fixes 1-3

- Lint/BooleanSymbol
- RSpec/BeEq
- Style/CaseLikeIf

(CONT-773) Rubocop Manual Fix 2 - RSpec/ExampleWording

(CONT-773) Set toapt_key `open` calls to `URI.open`

Calls to `open` have previously been left vague, which has caused issues upon implementing Ruby 3

(CONT-773) Rubocop Auto Fixes 16-17

- Style/RescueStandardError
- Style/TrailingCommaInHashLiteral

(CONT-773) Rubocop Auto Fixes 11-15

- RSpec/ExampleWording
- RSpec/ImplicitSubject
- Style/IfUnlessModifier
- Style/QuotedSymbols
- Style/RedundantRegexpEscape

(CONT-773) Rubocop Auto Fixes 6-10

- Lint/SymbolConversion
- RSpec/BeNil
- RSpec/EmptyLineAfterExample
- RSpec/EmptyLineAfterExampleGroup
- RSpec/EmptyLineAfterHook

(CONT-773) Rubocop Auto Fixes 1-5

- Layout/ClosingHeredocIndentation
- Layout/EmptyLineAfterGuardClause
- Layout/HashAlignment
- Layout/HeredocIndentation
- Layout/SpaceAroundMethodCallOperator

(CONT-773) Rubocop Manual Fix 1 - RSpec/MultipleExpectations

(CONT-773) Update Rubocop Versions

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

(CONT-773) Re-pin puppet_agent

(CONT-773) Add Support for Puppet 8 / Remove Support for Puppet 6

(CONT-773) PDK Update

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

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

(CONT-844) - Update .sync.yml

Merge pull request #1096 from puppetlabs/pdksync_CONT-229-Implement_reusable_workflows

pdksync - CONT-229-Implement reusable workflows

(CONT-229) - Implement reusable workflows

Merge pull request #1095 from puppetlabs/pdksync_(MAINT)_Pin_puppetlabs-puppet_agent

pdksync - (CONT-768) 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 #1092 from puppetlabs/release-prep

Release prep v9.0.2

Release prep v9.0.2

Merge pull request #1091 from puppetlabs/maint-pdk_update

(maint) - pdk update

(maint) - pdk update

Merge pull request #1090 from tuxmea/fix_pin_template

Adopt new parameter defaults in template

the apt::pin define has switched from empty string to Undef.
But the template was not updated.

fixes #1089

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

pdksync - (CONT-211) Adding mend config

Adding mend file

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

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

(CONT-494) PDK Update

(CONT-494) Pin github_changelog_generator gem version

Merge pull request #1085 from puppetlabs/CONT-493-ppa_validation_fix

(CONT-493) PPA validation adjustment

Merge pull request #1075 from xepa/xepa-patch-1

Updated mark as title can contain dot (fixes #1074)

(CONT-493) PPA validation adjustment

Prior to this commit, one of our updates (https://github.com/puppetlabs/puppetlabs-apt/pull/1052)
implemented a regex validation for ppa packages that were to be
installed. However, this validation did not account for resource
names that were dotted.

This commit aims to fix this bug in our validation process so that it
works as intended.

Update spec files

Updated regex after carefull reading of policy

Merge branch 'main' into xepa-patch-1

Merge pull request #1081 from johanfleury/fix/remove-apt-prefix-from-facts-variables

fix: remove `apt::` prefix from fact variables

Merge pull request #1082 from bastelfreak/foo

fix typo in source.pp

fix typo in source.pp

fix: remove `apt::` prefix from fact variables

Fact variables are top-scopped and should not be prefixed by module name.

This bug was introduced by cb6e58cedbd6d7a9bfc63c97c83d51eb39e9c7dc.

Updated regex in mark.pp

Now using debian policy manual as reference

Merge branch 'main' into xepa-patch-1

Merge pull request #1080 from puppetlabs/release-prep

Release prep v9.0.1

Merge branch 'main' into xepa-patch-1

Release prep v9.0.1

Merge pull request #1079 from puppetlabs/bugfix-bump_minimum_puppet_version

(bugfix) - Declare minimum Puppet version 6.24.0

(bugfix) - Declare minimum Puppet version 6.24.0

In codebase hardening efforts the commands are passed as an array, but this feature was only introduced in Puppet 6.24.01. This raises the minimum version to match, since it's no longer possible to use the module on anything older.

Merge branch 'main' into xepa-patch-1

Merge pull request #1076 from puppetlabs/pdksync_maint-remove_stale_bot

pdksync - (MAINT) Remove stalebot

(MAINT) Remove stalebot workflow

Updated mark as title can contain dot

Merge pull request #1069 from Programie/main

Do not remove PPA sources.list.d files if purge is enabled

Exec resource only exists on first run

Make sure add-apt-repository is executed before managing file resource for it

Ensure sources.list.d file resource exists so purge does not remove files created by apt::ppa

Merge pull request #1068 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

Fix rubucop linting error

This commit corrects an error identified by rubocop in spec testing.

Merge pull request #1067 from puppetlabs/pdksync_pdksync/remove_puppet_module_gems

pdksync - (PDKSync) Removal of puppet_module_gems

(PDKSync) Removal of puppet_module_gems

Merge pull request #1065 from puppetlabs/pdksync_remove_debian_9

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

Dropping Debian 9 Support

Merge pull request #1058 from puppetlabs/issue-1057

Add tests for valid/invalid resource names

Prior to this commit, ppa_spec.rb did not test the recently implemented
validation for resource names.

This commit aims to implement some test cases to make sure that valid
resource names are allowed while invalid or malicious resource names do
not work.

(GH-1057) Regex fix to allow dotted resources

Prior to this commit, one of our recent module updates introduced a
regex validation step for the resource names in our ppa.pp manifest
which would raise an issue if a valid resource name contained a dot (.).

This commit aims to slightly adjust the regex validation so that it
allows for dotted resource names. This PR should fix issue #1057.

Merge pull request #1056 from puppetlabs/gh-1055-hardcoded_cache_path

(GH-1055) Fix hardcoded cache path

Merge pull request #1053 from david22swan/GH-cat-9

(GH-cat-9) Update module to match current syntax standard

(GH-cat-9) Update module to match current syntax standard

Module is now in compliance with the following rules:
- optional_default
- strict_indent
- unquoted_string_in_case
- parameter_documentation
- relative_classname_inclusion
- no-top_scope_facts-check
- no-top_scope_variable-check
- variable_scope

The below exception has been left in place:
- disable_anchor_resource

(GH-1055) Fix hardcoded cache path

Prior to this commit the cache path used to create the script file resource
was hardcoded to /opt/puppetlabs/puppet/cache.

This commit fixes that by using the `puppet_vardir` fact provided by stdlib so
that we will always get the correct path for the OS that is executing the code.

Additionally, if for some reason the `puppet_vardir` fact is not available we
will fall back to `tmp`.

Merge pull request #1054 from puppetlabs/release-prep

Release prep v9.0.0

Release prep v9.0.0

Merge pull request #1052 from puppetlabs/maint-harden_ppa

Harden PPA defined type

Add spec test for add-apt-repository

This commit adds spec tests that validate the changes made in
the previous commit.

Harden PPA defined type

Prior to this commit there was a possibility that malformed strings
could be passed as the resources name. This could lead to unsafe
executions on a remote system.

This was also a possibility for the options parameter as it was
constrained to a string.

In addition, commands were not properly broken out in to arrays of
arguments when passed to the exec resource.

This commit fixes the above by adding validation to the resource name
ensuring that the given ppa name conforms to expectation. Also, commands
are now broken down in to arrays of arguments appropriately. This ensures
safer execution on the remote system.

Given that the options parameter, passed as a raw string, could lead to
unsafe code execution it was reasonable to change the accepted type to
an `Optional[Array[String]]. This means that an array of options can now
be passed to the exec resource inside the original command.

Merge pull request #1050 from ekohl/deal-with-net-ftp-as-default-gem

Deal with net-ftp being unavailable

Merge pull request #1051 from puppetlabs/maint-move_apt_mark_to_provider

Harden apt-mark defined type

Add spec tests for apt-mark

This commit adds additional spec tests for mark.pp.

The tests validate the new resource name requirements
introduced in the previous commit.

Harden apt-mark defined type

Prior to this commit the title parameter of this defined
type was not properly validated. This means that it could have been
possible to use a resource title outside of the normal bounds of
a package name.

Additionally the `onlyif` and `command` parameter values were
interpolated strings meaning that it may have been possible to
execute unsafe code on the remote system.

This commit fixes the above issues by adding a regex to check that the
resource title is a valid apt package name and also breaks out the
`onlyif` and `command` parameter values in to arrays of args ensuring
that the commands executed in a safe manor on the remote system.

The exception in this commit is the `unless_cmd`. This has not been
broken out in to an array of args due to the requirement of the command.
This is a reasonable trade of however due to the fact that action is
created from known enum values and title would be pre-validated.
This is also explained in mark.pp:20.

Deal with net-ftp being unavailable

In Ruby 3.0 net-ftp changed from a bundled gem to a default gem. This
means it may not be available, such as when running unit tests.

Since ftp is becoming less and less common, this changes net-ftp to be
an optional dependency. Users who do need ftp support should ensure the
gem is installed.

Merge pull request #1046 from puppetlabs/pdksync_GH-cat-11/main/add_ubuntu_22.04_support

pdksync - (GH-cat-11) Certify Support for Ubuntu 22.04

(GH-cat-11) Certify Support for Ubuntu 22.04

Merge pull request #1044 from david22swan/release_prep

Release prep v8.5.0