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

fix bug: KeyError for description_format #768

Closed
wants to merge 1 commit into from

Conversation

Brightcells
Copy link

@Brightcells Brightcells commented Aug 11, 2016

redis.StrictRedis(connection_pool=redis.ConnectionPool(**connect_settings))

When connect_settings doesn't have key 'host', 'port', or 'db', will raise KeyError:

@gyermolenko
Copy link
Contributor

it is hard to tell from description but if description_format from title is this one
than this is no longer an issue - all format arguments are passed to __init__ with defaults.

@andymccurdy
Copy link
Contributor

I believe this had to do with the repr() of Connection/ConnectionPool/Redis instances which has been fixed since 3.0. Closing this for now. If there's still an issue with this, please feel free to reopen.

@andymccurdy andymccurdy closed this Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants