Opened 3 years ago

Closed 3 years ago

#32301 closed Cleanup/optimization (fixed)

Raise CommandError when clearsessions is called on a backend not implementing clear_expired()

Reported by: Jacob Walls Owned by: François Freitag
Component: contrib.sessions Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Formal ticket for the issue raised by François Freitag in PR.

clearsessions writes to stderr directly when a backend has not implemented clear_expired(). François notes that the recommended behavior is to raise CommandError.

Change History (3)

comment:1 by Jacob Walls, 3 years ago

comment:2 by Mariusz Felisiak, 3 years ago

Triage Stage: UnreviewedAccepted

comment:3 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

Resolution: fixed
Status: assignedclosed

In b11ec9a6:

Fixed #32301 -- Made clearsessions raise CommandError when clear_expired() is not implemented.

Note: See TracTickets for help on using tickets.
Back to Top