Project

General

Profile

Bug #12606

Disable PHP sessions for Custom Sidebars plugin

Added by Raymond Hoh over 1 year ago. Updated over 1 year ago.

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

0%

Estimated time:

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.

History

#1 Updated by Boone Gorges over 1 year ago

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

#2 Updated by Raymond Hoh over 1 year ago

  • Status changed from New to Staged for Production Release

#3 Updated by Boone Gorges over 1 year ago

  • Status changed from Staged for Production Release to Resolved

Also available in: Atom PDF