(PUP-2967) Remove the deprecated recurse => inf

The use of 'inf' as a synonym for 'true' made sense when the 'recurse'
parameter could take integer values, instead of using 'recurselimit'.
But integer values for 'recurse' are no longer valid, so 'inf' makes
no sense.
(PUP-3019) Make PMT check for string status codes when downloading

Previously PMT was checking for integer status codes when downloading
modules. This results in all module downloads failing. After this PMT
will check for string status codes when downloading modules.
Merge pull request #2945 from adreyer/PUP-3019

(PUP-3019) Make PMT check for string status codes when downloading
Merge pull request #2940 from kylog/issue/pup-2967/remove-recurse-inf

(PUP-2967) Remove the deprecated recurse => inf
Merge branch 'master' into puppet-4
(PUP-2950) Rubocop Style/AndOr automatic pass

This commit replaces `and` and `or` keywords in favor of `&&` and `||` where it
can be done automatically (In places where the semantic meaning of the
expression does not change on replacing the `and` keyword with the operator `||`).

Since we are using `rubocop -a` to make the changes, we are assured of the
correctness of translation. These changes have also been individually verified.