Opened 17 years ago

Closed 17 years ago

#5548 closed (fixed)

os.getpid() workaround for jython missing on the recent session refactoring

Reported by: leosoto <leo.soto@…> Owned by: nobody
Component: Uncategorized Version: dev
Severity: Keywords: jython
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The "old" code for generating session ids was included into django/contrib/sessions/backends/base.py, without the workaround for Jython introduced on [6270].

Attachments (1)

os_pid_again.patch (1.0 KB ) - added by leosoto <leo.soto@…> 17 years ago.
patch

Download all attachments as: .zip

Change History (3)

by leosoto <leo.soto@…>, 17 years ago

Attachment: os_pid_again.patch added

patch

comment:1 by leosoto <leo.soto@…>, 17 years ago

By the way, SessionBase._get_new_session_key is a exact copy of SessionManager.get_new_session_key. Is it OK for some reason?

comment:2 by Adrian Holovaty, 17 years ago

Resolution: fixed
Status: newclosed

(In [6386]) Fixed #5548 -- Reintroduced Jython workaround for os.getpid(), which was lost in [6270]. Thanks, leosoto

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