Comparison error in apache::mod::php when RedHat and PHP 8

[polatsinan]

Describe the Bug

When $php_version is set to 8.0 and the OS is RedHat, the wrong php library is being loaded due to an incorrect comparison.

It should load libphp.so, but instead it tries to load libphp8.0.so.

The problem is in line 96:

puppetlabs-apache/manifests/mod/php.pp

Lines 92 to 101 in 6bc1072

	$_php_major = regsubst($php_version, '^(\d+)\..*$', '\1')
	$_php_version_no_dot = regsubst($php_version, '\.', '')
	if $apache::version::scl_httpd_version {
	$_lib = "librh-php${_php_version_no_dot}-php${_php_major}.so"
	} elsif ($facts['os']['family'] == 'RedHat') and ($_php_major == 8) {
	# RedHat + PHP 8 drop the major version in apache module name.
	$_lib = "${libphp_prefix}.so"
	} else {
	$_lib = "${libphp_prefix}${php_version}.so"
	}

Line 92 returns a string (so $_php_major is a string), but in line 96 it is comparing it to an integer.

A fix could be replacing line 96 with (adding single quotes to 8:
} elsif ($facts['os']['family'] == 'RedHat') and ($_php_major == '8') {
or even replace it with:
} elsif ($facts['os']['family'] == 'RedHat') and (versioncmp($php_version, '8') >= 0) {
like it is done on line 47:

puppetlabs-apache/manifests/mod/php.pp

Line 47 in 6bc1072

if ($facts['os']['family'] == 'RedHat') and (versioncmp($php_version, '8') >= 0) {

I can create a PR if the bug is accepted as a bug.

Expected Behavior

The correct php library (libphp.so) should be loaded when using RedHat + PHP 8.

Steps to Reproduce

Steps to reproduce the behavior:

1. Set $php_version to 8.0 in a RHEL environment.

Environment

• Puppetlabs/apache 8.1.0
• RHEL 8.6