Page MenuHomePhabricator

respect $exptime in RESTBagOStuff::incr()
Open, Needs TriagePublic

Description

from https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/core/+/master/includes/libs/objectcache/RESTBagOStuff.php#154

I don't see $exptime in the signature for BagOStuff::incr(), so I don't know how this should be handled. Also, this is not important for session storage, which I think would not use this method.

per @aaron

Event Timeline

We don't need this for using RESTBagOStuff with Kask for sessions, so I'm taking it out of the epic and removing the session tag.

It does, however, need to be associated with some project on the Core Platform Team workboard.

Removing myself as assignee because I am not currently working on this, and am not scheduled to in the near future. I would not object to working on it on the future if plans and priorities allow, but I do not want to give a false impression of status or discourage anyone else from taking it over.