Opened 8 years ago

Closed 8 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: master
Severity: Keywords: jython
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

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@…> 8 years ago.
patch

Download all attachments as: .zip

Change History (3)

Changed 8 years ago by leosoto <leo.soto@…>

patch

comment:1 Changed 8 years ago by leosoto <leo.soto@…>

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

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 Changed 8 years ago by adrian

  • Resolution set to fixed
  • Status changed from new to closed

(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