ssandison

I’ve already patched the plugin now but looking at it more closely this is to do with the plugin being compatible with WP 5.9 onwards and this is a site where I haven’t updated it yet.

WP 5.9 adds this function in for sites that are using PHP below version 8 so I guess its ok if you have 5.9 onwards.

Your plugin minimum WordPress version is 5.9 so this error makes sense now.

Yes I have the same issue in mobile edge. I think it is could the cookies are being constructed. I updated the code as follows but you need PHP 7.3:

function samesite_setcookie($name, $value, array $options) {

	setcookie($name, $value, $options);
	$_COOKIE[$name] = $value;

/*
	$header = 'Set-Cookie:';
	$header .= rawurlencode($name) . '=' . rawurlencode($value) . ';';

	if (!empty($options['expires']) && $options['expires'] > 0) {
		$header .= 'expires=' . \gmdate('D, d-M-Y H:i:s T', (int) $options['expires']) . ';';
		$header .= 'Max-Age=' . max(0, (int) ($options['expires'] - time())) . ';';
	}

	$header .= 'path=' . rawurlencode($options['path']). ';';
	$header .= 'domain=' . rawurlencode($options['domain']) . ';';

	if (!empty($options['secure'])) {
		$header .= 'secure;';
	}
	$header .= 'httponly;';
	$header .= 'SameSite=' . rawurlencode($options['samesite']);

	header($header, false);
	$_COOKIE[$name] = $value;
*/

}

Hi Peter,

Apologies for the delay in getting this to you. I have just sent you over an example now.

Cheers,

Steve

Thanks Peter, I shall get in touch.

Hi Peter,

No I mean’t that there is only one database but multiple connection credentials for different environments. It’s basically a switch statement based on hostname that defines the correct constants for the environment the site is in (dev, UAT and production). There is only one wordpress database that we are using.

The issue is that wordfence seems to read the wp-config.php file as if it were a text file and parse it to get the db connection credentials but is unaware of the switch statement so uses the last set of credentials.

Are you not able to cache the credentials in a config file similar to how you are doing with other wflogs config files?

Cheers,

Steve

• This reply was modified 3 years, 8 months ago by ssandison.
• This reply was modified 3 years, 8 months ago by ssandison.
• This reply was modified 3 years, 8 months ago by ssandison.

Perfect thanks

Yes it would be good to know what’s going on. Do we need to disable this plugin? If there is an issue does it affect the latest version and not older versions etc?

Hi,

Sorry for the delay in responding. This is definately something I have thought of adding. I need it myself to be honest. I will work on this feature asap.

Cheers,

Steve

I used the following:

function start_el(&$output, $item, $depth=0, $args=array()) {

	// Has children
	$children = get_posts(array('post_type' => 'nav_menu_item', 'nopaging' => true, 'numberposts' => 1, 'meta_key' => '_menu_item_menu_item_parent', 'meta_value' => $item->ID));
	if (empty($children)) {
		$classes[] = 'no_children';
	}

	[...]

Hope this is of use.

Sorry donncha I forgot all about this issue because I was working on something else but I will have a go at debugging and will also check to see if I get the same problem with an unmodified code base.

Cheers,

Steve