(MODULES-7023) - Removing duplication in sync.yml

Merge pull request #748 from bernhardschmidt/patch-1

Replace UTF-8 whitespace in comment

Replace UTF-8 whitespace in comment

There is an UTF-8 whitespace which blows up older puppet-lint versions. Replace with space character

./modules/apt/manifests/source.pp
/usr/lib/ruby/vendor_ruby/puppet-lint/checks.rb:23:in `split': invalid byte sequence in US-ASCII (ArgumentError)
from /usr/lib/ruby/vendor_ruby/puppet-lint/checks.rb:23:in `load_data'
from /usr/lib/ruby/vendor_ruby/puppet-lint/checks.rb:53:in `run'
from /usr/lib/ruby/vendor_ruby/puppet-lint.rb:174:in `run'
from /usr/lib/ruby/vendor_ruby/puppet-lint/bin.rb:59:in `block in run'
from /usr/lib/ruby/vendor_ruby/puppet-lint/bin.rb:56:in `each'
from /usr/lib/ruby/vendor_ruby/puppet-lint/bin.rb:56:in `run'
from /usr/bin/puppet-lint:7:in `<main>'
Detected error

Merge pull request #747 from aboks/patch-1

Fix "E: Unable to locate package  -y"

Fix "E: Unable to locate package  -y"

Due to the extra space, apt complained with "Unable to locate package  -y". Without the extra space, the task runs fine.

Merge pull request #743 from smortex/fix-automatic-coercion-warning

Fix automatic coercion warning

Merge pull request #745 from puppetlabs/cve_release

4.5.1 mergeback

Merge pull request #2 from puppetlabs/cve-2018-6508

(RE-10220) Fix CVE-2018-6508

(RE-10220) Fix CVE-2018-6508

Do not rely on deprecated argument signature

This commit fixes the following warning in the puppet master's logs:

Jan 26 16:09:26 <daemon.warn> marvin puppet-master[68168]: The argument signature (String format, [String timezone]) is deprecated for #strfime. See #strftime documentation and Timespan type for more info
Jan 26 16:09:26 <daemon.warn> marvin puppet-master[68168]:    (at /usr/local/etc/puppet/environments/production/modules/apt/manifests/update.pp:17)

Fix automatic coercion warning

Explicitly cast the return value of strftime() to an Integer to avoid
spamming the puppet server log file with warning messages:

Jan 26 14:19:13 <daemon.warn> marvin puppet-master[27147]: The string '1516972753' was automatically coerced to the numerical value 1516972753 at /usr/local/etc/puppet/environments/production/modules/apt/manifests/update.pp:17:27
Jan 26 14:19:25 <daemon.warn> marvin puppet-master[27147]: The string '1516972765' was automatically coerced to the numerical value 1516972765 at /usr/local/etc/puppet/environments/production/modules/apt/manifests/update.pp:17:27
Jan 26 14:19:55 <daemon.warn> marvin puppet-master[27147]: The string '1516972795' was automatically coerced to the numerical value 1516972795 at /usr/local/etc/puppet/environments/production/modules/apt/manifests/update.pp:17:27
Jan 26 14:20:15 <daemon.warn> marvin puppet-master[27147]: The string '1516972815' was automatically coerced to the numerical value 1516972815 at /usr/local/etc/puppet/environments/production/modules/apt/manifests/update.pp:17:27

Merge pull request #742 from puppetlabs/release

Release Mergeback 4.5.0

Merge pull request #741 from pmcmaw/pdk_convert-release

(MODULES-6476) - Release prep apt 4.5.0

(MODULES-6476) - Release prep apt 4.5.0

Merge pull request #740 from pmcmaw/pdk_convert

(MODULES-6452) - PDK Convert apt

Remove update bundler

(MODULES-6452) - PDK Convert apt

Using PDK to convert module so that it is compliant with the PDK.

The second commit addresses a bundler issue MODULES-6339.
The next time PDK convert is ran it will re-add the newer version of the bundler gem.

Merge pull request #739 from pmcmaw/puppet-lint_warn

(maint) - Addressing puppet-lint doc warnings

(maint) Addressing puppet-lint doc warnings

Carrying this out in order to prep for making this module PDK compliant.

Merge pull request #738 from puppetlabs/msync_MODULES-6339

(maint) modulesync 65530a4 Update Travis

(maint) modulesync 65530a4 Update Travis

Related: https://github.com/puppetlabs/modulesync_configs/pull/177

Merge pull request #737 from willmeek/modulesync_rubocop

Modulesync rubocop

Fixes rubocop issues found by modulesync

(maint) - modulesync 384f4c1

Merge pull request #735 from pmcmaw/rubocop

MODULES-6235 - Addressing Rubocop Errors

Addressing Rubocop Errors

Merge pull request #734 from puppetlabs/release

Release merge back 4.4.1

Merge pull request #733 from puppetlabs/maint_modsync_1d81b6a

(maint) - modulesync 1d81b6a

Module sync 1d81b6a

Merge pull request #732 from willmeek/release_4_4_1

Release prep for 4.4.1

Release prep for 4.4.1

Merge pull request #731 from willmeek/rubocopification

Rubocopification

Rubocopification

Merge pull request #730 from puppetlabs/release

Release mergeback 4.4.0

Merge pull request #729 from jbondpdx/release

fix inconsistencies in README

fix inconsistencies in README

Merge pull request #728 from willmeek/release_4_4_0_a

Further release prep for 4.4.0

Further release prep for 4.4.0

Release prep for 4.4.0

Add a check for Puppet version to task helper

This commit will add a check for Puppet version being 5 or greater
around the task test helper. This ensures that is it not attempted
pre-task versions of puppet.

Merge pull request #724 from wyardley/apt_transport_https2

install apt-transport-https on Debian 7 as well

Merge pull request #723 from wyardley/docs_fix_numbering

Fix numbering in README index

install apt-transport-https on Debian 7 as well

use 1. for numbers so the numbers don't start at 2

Merge pull request #719 from willmeek/add_dist_fact

Add a facter fact for dist-upgrade

Add a facter fact for dist-upgrade

This commit adds a facter fact for dist-upgrade, along with
associated facter facts.

Merge pull request #718 from willmeek/http_proxy_bypass

Http proxy bypass

Merge pull request #720 from actatux/apt-transport-https

Install apt-transport-https if needed

Add a 'direct' option to proxy settings

This commit adds a 'direct' boolean option to proxy settings.
When set to true, if https is not true, the https proxy is set
to 'DIRECT'.

Install apt-transport-https if needed

Without this package, the module fails to update a repository with a
location using the https method.

This extends to Debian stretch, Ubuntu trusty and xenial.

https support has been moved into the apt package for future releases
of these distibutions, thus this change won't need to be updated.

https proxy bypass fix

* if http proxy is set without an https proxy apt will still attempt to
proxy those https sources, resulting in errors.  This fix will allow
direct connect to the https urls, bypassing the http proxy.  This is the
most simple fix for this problem, a more complex fix would be to have an
https_direct setting.
fix test for https proxy bypass

Merge pull request #717 from willmeek/build_fix

Remove tasks acceptance test for non-Debian builds

Remove tasks acceptance test for non-Debian builds

This commit will disable the task test on non-Debian/Ubuntu based masters.

Merge pull request #716 from kbarmen/master

Do not treat debian stable-updates as security updates

Output from apt-get with actual security updates

The previous output just had debian-updates and backports, none
of which are security updates. Adjusted expected results accordingly,
using curl as an example.

debian-updates is not security updates

Debian-updates are pending packages between dot-releases,
they are not security updates. When Debian has a dot-release,
for example 8.7, debian-updates is empty. Between 8.7 and 8.8,
updates packages go to debian-updates, and on 8.8, all packages
in debian-updates are moved (or replaced) by packages in the main
repo, leaving debian-updates empty again. Security updates are
managed outside of this.

Merge pull request #714 from actatux/apt-transport-https_debian8

Install apt-transport-https in Debian 8 if needed

Merge pull request #710 from TMGMedia/circular-dep-fix

Fixed circular dependency for package dirmngr

Install apt-transport-https in Debian 8 if needed

Without this package, the module fails to update a repository with a
location using the https method.

Merge pull request #711 from icann-dns/fix_legacy_functions

remove legacy functions

Merge pull request #713 from willmeek/update_release_link

Update link to Release in CHANGELOG

Update link to Release in CHANGELOG

Merge pull request #712 from puppetlabs/release

4.3.0 merge back to master

FM-6474 Adding an Apt task.

Remove apt::key was wrong, we only need to remove the requirement

It is not really elegant to have them seperated, but tying them together
will create a circular dep with init.pp.

Signed-off-by: Henry <h.paulissen@tmg.nl>

remove legacy functions

Fixed circular dependency for package dirmngr

Signed-off-by: Henry <h.paulissen@tmg.nl>

Merge pull request #706 from tphoney/rubocopification

Cleanup ruby code via rubocop.

Cleanup via rubocop of ruby code

Merge pull request #651 from nhinds/MODULES-4265-ubuntu-security-updates

(MODULES-4265) Detect security updates from multiple sources

(MODULES-4265) Detect security updates from multiple sources

The apt_security_updates and apt_package_security_updates facts
now detect security updates which come from more than one source
repository.

Merge pull request #709 from puppetlabs/release

(MODULES-5628) 4.2.0 release merge back

Merge pull request #708 from eputnam/release

(maint) update changelog entry for support/compat clarification [skip ci]

(maint) update changelog entry for support/compat clarification [skip
ci]

Merge pull request #707 from puppetlabs/jbondpdx-patch-1

minor edits to new material

minor edits to new material

Merge pull request #657 from puppetlabs/binford2k-patch-1

This class should be private

Merge pull request #705 from HAIL9000/maint/release/MODULES_5628_apt_4.2.0_release_prep

(MODULES-5628) 4.2.0 release prep

Update update.pp

(MODULES-5628) 4.2.0 release prep

Merge pull request #704 from puppetlabs/release

(maint) Update release merge back

Merge pull request #703 from tphoney/jle64-patch-1

apt_package_security_updates fact and test

adding unit tests for new fact

Add apt_package_security_updates fact

Merge pull request #702 from tphoney/raoulbhatia

Switch to deb.debian.org and remove Debian 6.0

Merge pull request #700 from deric/outdated-doc

Delete documentation for parameters that already has been removed

Switch to deb.debian.org and remove Debian 6.0

Merge pull request #696 from trevorrea/master

Add 16.04 to list of distributions that should use software-properties-common.

Merge pull request #701 from puppetlabs/msync_18sep17_892c4cf

(maint) modulesync 892c4cf

Merge pull request #697 from aeber/master

fix documentation for parameter $proxy['port'] in class apt

(maint) modulesync 892c4cf

Delete documentation for parameters that already has been removed

Merge pull request #698 from deric/MODULES-4686-fix

MODULES-4686: gpg keyserver import fails in Debian 9 (Stretch)

fix apt-key resource syntax

autorequire dirmngr package

* add Debian 9 as supported system
* move dirmngr ensure to init.pp

support ubuntu

ensure dirmngr is installed on Debian Stretch

(MODULES-4686) fix incorrect GPG keys parsing on Debian 9

patch from Alex P (alexpr)

fix documentation for parameter $proxy['port'] in class apt

Documentation says 'port' should be a string. init.pp checks for type
integer.

Merge pull request #665 from tigercomputing/master

Ignore subkeys in apt-key's output