Django

Code

Ticket #5548 (closed: fixed)

Opened 1 year ago

Last modified 1 year ago

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

Reported by: leosoto <leo.soto@gmail.com> Assigned to: nobody
Milestone: Component: Uncategorized
Version: SVN Keywords: jython
Cc: Triage Stage: Unreviewed
Has patch: 1 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

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

os_pid_again.patch (1.0 kB) - added by leosoto <leo.soto@gmail.com> on 09/19/07 21:04:44.
patch

Change History

09/19/07 21:04:44 changed by leosoto <leo.soto@gmail.com>

  • attachment os_pid_again.patch added.

patch

09/19/07 21:09:20 changed by leosoto <leo.soto@gmail.com>

  • needs_better_patch changed.
  • needs_tests changed.
  • needs_docs changed.

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

09/19/07 21:19:48 changed by adrian

  • status changed from new to closed.
  • resolution set to fixed.

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

Add/Change #5548 (os.getpid() workaround for jython missing on the recent session refactoring)




Change Properties
Action