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

Release prep v8.5.0

Merge pull request #1042 from david22swan/GH-1038/main/check_valid_until

(GH-1038) add support for `check-valid-until` configuration

(GH-1038) add support for `check-valid-until` configuration

Add's additional configuration to `apt::source` to allow the user to specify whether or not to check if  the repository that they are accessing has a valid release ate.
Defaults to `True`

Merge pull request #1040 from david22swan/release_prep

Release prep v8.4.1

Release prep v8.4.1

Merge pull request #1039 from david22swan/GH-1036/main/dependency_cycle

(ISSUE-1036) Conditional `gnupg` include added to init.pp

(ISSUE-1036) Conditional `gnupg` include added to init.pp

Originally removed as it was causing `gnupg` to be installed in all OS when it wasn't needed, removing it seems to have caused a dependency cycle in the relevant Debian family OS for certain community members.
Adding the include back within a conditional statement to solve the issue while still preventing it from being included when unneeded.

Merge pull request #1035 from puppetlabs/Release_prep-v8.4.0

Release prep v8.4.0

Syntax

Release prep v8.4.0

Merge pull request #1032 from puppetlabs/pdksync_maint/pdk_update_stalebot

pdksync - (MAINT) Stale bot config/msg update

Config update

(MAINT) Stale-bot config/msg update

Merge pull request #1030 from puppetlabs/pdksync_maint/pdk_update_20-04

pdksync - (Maint) PDK Update

(GH-cat-9) syntax:hiera:yaml exclusions added

(GH-cat-9) syntax:hiera:yaml fixes

(maint) PDK Update

Merge pull request #1029 from puppetlabs/pdksync_pdksync-use_pull_request_target

pdksync - PDKSYNC Update labeller trigger

"This commit changes the workflow trigger for pull requests to pull_request_target"

Merge pull request #1019 from puppetlabs/pdksync_pdksync_heads/main-0-gf3911d3

pdksync - pdksync_heads/main-0-gf3911d3

(GH-C&T-9) Temporarily disable syntax checks

Merge pull request #1020 from simondeziel/optional-gnupg

(MODULES-11301) Don't install gnupg if not needed

Merge pull request #1025 from puppetlabs/pdksync_pdksync_add_workflows

pdksync - Add labeller and stale GHA workflows

(MAINT) Fixes no new line at EOF

(MAINT) Add labeller and stale GHA workflows

Merge pull request #1024 from david22swan/GH-iac-334/main/remove_compatible_code

(GH-iac-334) Remove code specific to unsupported OSs

(GH-iac-334) Remove code specific to unsupported OSs

Code removed includes ubuntu 16.04 + 14.04 and Debian 7 + 8

Merge pull request #1023 from puppetlabs/pdksync_GH-iac-334/main/remove_ubuntu_14.04_support

pdksync - (GH-iac-334) Remove Support for Ubuntu 14.04

(GH-iac-334) Remove Support for Ubuntu 14.04

Merge pull request #1022 from puppetlabs/pdksync_GH-iac-334/main/remove_ubuntu_16.04_support

pdksync - (GH-iac-334) Remove Support for Ubuntu 16.04

(GH-iac-334) Remove Support for Ubuntu 16.04

(MODULES-11301) Don't install gnupg if not needed

apt::key has the needed ensure_packages() to bring gnupg only
when needed.

Signed-off-by: Simon Deziel <simon@sdeziel.info>

pdksync_heads/main-0-gf3911d3

Merge pull request #1017 from root-expert/safe-facts

Switch using os.release.major for apt-transport-https

Signed-off-by: Christos Papageorgiou <christos.papageorgioy@gmail.com>

Merge pull request #1016 from puppetlabs/kreeuwijk-patch-1

Clarify this Task runs apt-get, not apt

Use fact() function for all os.distro.* facts

* On Puppet 6 facter 3.x requires lsb-release to resolve os.distro.* facts. Using $facts hash cause errors like "Evaluation Error: Operator '[]' is not applicable to an Undef Value." because os.distro is undefined causing the catalog to fail. Use fact() to identify Undef facts and throw an error to the user.

Signed-off-by: Christos Papageorgiou <christos.papageorgioy@gmail.com>

Clarify this Task runs apt-get, not apt

(maint) Fix resource ordering when apt-transport-https is needed (#1015)

Adds a dependency to ensure apt-transport-https is installed when the
repositories are updated.

enable allow-insecure for apt::source defined types, includes new tests, documentation (#1014)

Co-authored-by: Lukas Audzevicius <97180854+LukasAud@users.noreply.github.com>

Merge pull request #1013 from mpdude/patch-3

Omit empty options in source.list template to fix MODULES-11174

Merge pull request #1012 from mpdude/patch-1

Replace `arm64` for `aarch64` in `::apt::source`

Merge pull request #1011 from Conzar/ppa-ubuntu-fix

Fixed reading conditions for gpg file for Ubuntu versions 21.04 and later.

Omit empty options in source.list template to fix MODULES-11174

Empty `[name= ...]` options in a soures list lead to Apt parsing errors.

This change skips such empty options, resolving https://tickets.puppetlabs.com/browse/MODULES-11174.

Replace `arm64` for `aarch64` in `::apt::source`

Fixed gpg file for Ubuntu versions 21.04 and later.

As of Ubuntu 21.04, Canotical has again changed the filename of the gpg files going back to dashes
instead of using underscores.

Merge pull request #1010 from LTangaF/remove_frequency_collector

(MODULES-10763) Remove frequency collector

(MODULES-10763) Remove frequency collector
The case logic in apt::update adequately covers the 'always' case and
the collector causes issues in acceptance testing.

Merge pull request #1009 from puppetlabs/release-prep

Release prep v8.3.0

Release prep v8.3.0

Merge pull request #1008 from puppetlabs/pdksync_IAC-1598/main/remove_debian_8_support

pdksync - (IAC-1598) - Remove Support for Debian 8

Merge pull request #1007 from maturnbull/proxy_per_host

(MODULES-11173) Add per-host overrides for apt::proxy

(IAC-1598) - Remove Support for Debian 8

Merge pull request #999 from smortex/debian-frontend

(maint) Set DEBIAN_FRONTEND=noninteractive on upgrade

(main) Do not prompt on configuration file update

When updating configuration file, apt by default prompt the user for the
action to perform.  Since we are running in a non-interactive context,
skip these prompts:

  * `--force-confdef` ensure the configuration file is replaced by a
    newer version in the package if the previous package default
    configuration file was not modified;
  * `--force-confold` keep the configuration file as it is if it has
    been modified compared to the one shipped in the previous version of the
    package.

(MODULES-11173) Add per-host overrides for apt::proxy

Add support for optional apt::proxy::perhost array to configure bypass (DIRECT) or alternative proxy for specific repository hosts.

Merge pull request #1006 from puppetlabs/release-prep

Release prep v8.2.0

Merge pull request #1004 from puppetlabs/maint/main/remove_since_tag

(maint) - Correct since tag

Release prep v8.2.0

(maint) - Correct since tag

Merge pull request #1001 from smortex/debian11

(maint) Add support for Debian 11

Merge pull request #1000 from smortex/stdlib

(main) Allow stdlib 8.0.0

(maint) Add support for Debian 11

Debian 11 was recently released.  Update metadata.json to indicate that
this version is supported by the module.

(main) Allow stdlib 8.0.0

We recently released stdlib 8.0.0.  This module is comptatible with this
version of stdlib.

(maint) Set DEBIAN_FRONTEND=noninteractive on upgrade

When upgrading Debian packages, the system sometimes what to prompt the
user about what action to perform.  Since a tasks is supposed to be
non-interactive, we should disable such prompts.

This help when updating some packages, e.g. postfix.

Merge pull request #996 from david22swan/maint/main/max_changelog

(maint) - Set max_issues for changelog generator to 500

(maint) - Set max_issues for changelog generator to 500

Merge pull request #994 from puppetlabs/release

Release mergeback version 8.1.0

Add label for PR

(maint) - Update since_tag

Release version 8.1.0

Merge pull request #993 from kenyon/source_weak_ssl

apt::source: pass the weak_ssl param to apt::key

Merge pull request #991 from johanfleury/feature/add-support-for-signed-by-in-source-entries

[MODULES-9695] - Add support for signed-by in source entries

apt::source: pass the weak_ssl param to apt::key

Merge pull request #992 from puppetlabs/pdksync_iac-1149

pdksync - (maint) - Updating Contribution Guidelines

(maint) Updating CONTRIBUTING guidelines

Add support for signed-by in source entries

Merge pull request #990 from puppetlabs/apt_keyserver

Replace sks-keyservers.net as is deprecated

Replace sks-keyservers.net(deprecated) with keyserver.ubuntu.com

test

Repair changelog and history files

Merge pull request #988 from puppetlabs/pdksync_pdkupdate

pdksync - (maint) PDKSync - PDK Update

(maint) pdksync - PDK Update

Merge pull request #987 from pmcmaw/IAC-1597

(IAC-1597) Increasing MAX_RETRY_COUNT

(IAC-1597) Increasing MAX_RETRIES for Ubuntu18

Merge pull request #986 from puppetlabs/pdksync_pdk_update

pdksync - (maint) Running pdk update

(maint) Running a pdk update to consume latest pdk-template changes

Merge pull request #985 from kenyon/patch-2

apt::mark: fix docs

apt::mark: fix docs

* Add Puppet Strings summary.
* Typo fix in link to manual page.