Making Session model and SessionStore classes more easily extendable
|Reported by:||sergeykolosov||Owned by:||sergeykolosov|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||yes|
While developing a feature the involves adding an extra database field to the Session model (thus creating a new session backend based on Django's one), I faced the need to copy almost everything from models.py, backends/db.py and backends/cached_db.py.
The reason why one have to do that, is the way that Session model is imported and used inside backends/db.py and backends/cached_db.py, and the way SessionStore class imported and used inside models.py.
Since the custom backend is a part of a shared package, which is intended to be used with multiple versions of Django, it's challenging to do that given the above.
I suggest the following changes:
- extract an abstract base model from Session;
- make Session class a property of SessionStore class, removing hardcoded usages;
- make key_prefix a property of SessionStore, defaulting to KEY_PREFIX;
- (anything else to make building a custom backend based on Django's one easier).
Change History (6)
comment:1 Changed 10 months ago by russellm
- Needs documentation unset
- Needs tests unset
- Patch needs improvement unset
- Triage Stage changed from Unreviewed to Accepted
comment:2 Changed 10 months ago by sergeykolosov
- Owner changed from nobody to sergeykolosov
- Status changed from new to assigned