CODEWITHSUNDEEP
pythondjangoredisdjango-admin
hR 312
hR 312

Reputation: 854

Receiving "'hmset' with mapping of length 0" error

I want to store my session data on redis dataset. I have set SESSION_ENGINE = 'redis' in settings.py.

Code for redis.py

#redis.py
from django.contrib.sessions.backends.base import SessionBase
from django.utils.functional import cached_property
from redis import Redis

class SessionStore(SessionBase):
    @cached_property

    def _connection(self):
        return Redis(
            host='127.0.0.1',
            port='6379',
            db=0,
            decode_responses=True
        )

    def load(self):
        return self._connection.hgetall(self.session_key)


    def exists(self, session_key):
        return self._connection.exists(session_key)

    def create(self):
        # Creates a new session in the database.
        self._session_key = self._get_new_session_key()
        self.save(must_create=True)
        self.modified = True

    def save(self, must_create=False):
        # Saves the session data. If `must_create` is True,
        # creates a new session object. Otherwise, only updates
        # an existing object and doesn't create one.
        if self.session_key is None:
            return self.create()
        data = self._get_session(no_load=must_create)
        session_key = self._get_or_create_session_key()
        self._connection.hmset(session_key, data)
        self._connection.expire(session_key, self.get_expiry_age())

    def delete(self, session_key=None):
        # Deletes the session data under the session key.
        if session_key is None:
            if self.session_key is None:
                return
            session_key = self.session_key
        self._connection.delete(session_key)
    @classmethod

    def clear_expired(cls):
        # There is no need to remove expired sessions by hand
        # because Redis can do it automatically when
        # the session has expired.
        # We set expiration time in `save` method.
        pass

I am receiving 'hmset' with mapping of length 0 error on accessing http://localhost:8000/admin in django.

After removing SESSION_ENGINE='redis' I am not receiving this error.

Upvotes: 1

Views: 883

Answers (1)

Uros
Uros

Reputation: 61

From redis documentation:

As per Redis 4.0.0, HMSET is considered deprecated. Please use HSET in new code.

I have replaced this line in save() method:

self._connection.hmset(session_key, data)

with:

self._connection.hset(session_key, 'session_key', session_key, data)

On making the change, it works as expected.

Upvotes: 1

Related Questions