Project

General

Profile

Actions

Bug #12606

closed

Disable PHP sessions for Custom Sidebars plugin

Added by Raymond Hoh about 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
WordPress Plugins
Target version:
Start date:
2020-04-02
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

I see a bunch of errors in the log for the custom-sidebars plugin relating to PHP sessions:

[Thu Apr 02 02:05:35 2020] [warn] [pid 29576] sapi_apache2.c(351): [client 154.20.100.138:50413] PHP Warning:  session_start(): Failed to read session data: memcached (path: X:11211?persistent=1&weight=2&timeout=10&retry_interval=10) in /wp-content/plugins/custom-sidebars/inc/external/wpmu-lib/inc/class-thelib.php on line 145

I've disabled PHP sessions when testing locally with the 'wdev_lib-use_session' filter - https://github.com/cuny-academic-commons/cac/blob/1.16.x/wp-content/plugins/custom-sidebars/inc/external/wpmu-lib/inc/class-thelib.php#L117

And it appears that sessions are only used to display admin notices in the dashboard. During testing, even though the admin notices no longer showed up, the functionality of the plugin worked just fine.

This filter is only used in the custom-sidebars plugin so it should be safe to roll with turning sessions off.

Let me know what you think, Boone.

Actions #1

Updated by Boone Gorges about 4 years ago

Yeah, let's go ahead and disable via this filter across the Commons. Thanks!

Actions #2

Updated by Raymond Hoh about 4 years ago

  • Status changed from New to Staged for Production Release
Actions #3

Updated by Boone Gorges about 4 years ago

  • Status changed from Staged for Production Release to Resolved
Actions

Also available in: Atom PDF