martinrusev / django-redis-sessions

Session backend for Django that stores sessions in a Redis database
BSD 3-Clause "New" or "Revised" License
495 stars 106 forks source link

Session store should not directly set session_key #12

Closed petrounias closed 12 years ago

petrounias commented 12 years ago

In session.py:

def create(self):
    while True:
        self.session_key = self._get_new_session_key()

should be:

        self._session_key = self._get_new_session_key()
jeffbaier commented 12 years ago

I believe this was already fixed in https://github.com/martinrusev/django-redis-sessions/commit/f65701a3f7edbd59dff1c8af24775392e975a9ee

Unfortunately the version on PyPi does not have this fix.

petrounias commented 12 years ago

Any chance of updating the package on PyPi?

martinrusev commented 12 years ago

I pushed it to Pypi http://pypi.python.org/pypi/django-redis-sessions/0.3.0