silverstripe / silverstripe-session-manager

Allow users to manage and revoke access to multiple login sessions across devices.
BSD 3-Clause "New" or "Revised" License
9 stars 7 forks source link

New queued job #22

Closed brynwhyman closed 3 years ago

brynwhyman commented 3 years ago

Overview

Garbage collection was recently moved to a crontask for the session manager module, see https://github.com/silverstripe/silverstripe-session-manager/pull/9

This relies on the silverstripe/contask module being installed, which isn't a commonly used module, compared to symbiote/queuedjobs.

For all sites, a new queued job should be created and added on dev build to ensure the garbage collection tasks can happen without additional Developer intervention.

Acceptance Criteria

Notes

PRs

brynwhyman commented 3 years ago

From @maxime-rainville :

I was just looking at the install stats for crontask vs queued jobs ... according packagist, queued jobs and crontask are about as popular. If anything crontask seems like it's gaining in popularity, while queued job is stagnating.

My point is that my initial thought that we should prioritise Queued job over crontask was probably wrong.

On the note of: "Validate that the crontask isn't required", there are other modules that I expect suggest the use of queuedjobs but don't enforce it - we should see how they handle this. E.g.:

emteknetnz commented 3 years ago

CWP Kitchen Sink module usage of queuedjobs vs crontask where both are required. Analysis of composer.lock:

symbiote/silverstripe-queuedjobs

require

suggest:

silverstripe/crontask

require:

suggest: