Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DataError: Invalid input of type: 'CacheKey'. Convert to a byte, string or number first. #921

Closed
czlee opened this issue Nov 15, 2018 · 3 comments

Comments

@czlee
Copy link
Member

czlee commented Nov 15, 2018

https://sentry.io/tabbycat-ig/backend/issues/764632985/

DataError: Invalid input of type: 'CacheKey'. Convert to a byte, string or number first.
(26 additional frame(s) were not displayed)
...
  File "redis/client.py", line 1182, in exists
    return self.execute_command('EXISTS', *names)
  File "redis/client.py", line 754, in execute_command
    connection.send_command(*args)
  File "redis/connection.py", line 619, in send_command
    self.send_packed_command(self.pack_command(*args))
  File "redis/connection.py", line 659, in pack_command
    for arg in imap(self.encoder.encode, args):
  File "redis/connection.py", line 124, in encode
    "byte, string or number first." % typename)

DataError: Invalid input of type: 'CacheKey'. Convert to a byte, string or number first.
@czlee
Copy link
Member Author

czlee commented Nov 15, 2018

Probably related: jazzband/django-redis#342

czlee added a commit that referenced this issue Nov 15, 2018
This is a stopgap for a compatibility issue relating to a
dependency of a dependency (redis-py, which is a requirement of
django-redis). See:
jazzband/django-redis#342
@czlee
Copy link
Member Author

czlee commented Nov 15, 2018

7bb5ce4 provides a workaround, but we should leave this open until the PR in django-redis is merged in, and then we've upgraded django-redis in our requirements file (and removed the pinned version for redis) accordingly.

@tienne-B
Copy link
Member

The Django-redis PR is merged and upgraded in the requirements file. The version of redis has also been updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants