Skip to content

Added support for acl authentication.#31

Merged
chayim merged 7 commits intomasterfrom
acl_support
Feb 24, 2022
Merged

Added support for acl authentication.#31
chayim merged 7 commits intomasterfrom
acl_support

Conversation

@MeirShpilraien
Copy link

Fix #30, added support for acl authentication.

pyproject.toml Outdated
[tool.poetry.dependencies]
python = "^3.6"
redis = "3.5.3"
redis = "4.1.3"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

redis = "^4.1.3"

This keeps it within the same minor patch set - and we've already released 4.1.4

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

tox.ini Outdated
commands_pre =
pip install --upgrade pip
{envdir}/bin/python -m pip install poetry
{envdir}/bin/python -m pip install RLTest==0.5
Copy link
Contributor

@chayim chayim Feb 22, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"RLTest~=0.5.0"

Isn't this to keep within the same patch set?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@chayim chayim merged commit 7707bb4 into master Feb 24, 2022
@chayim chayim deleted the acl_support branch February 24, 2022 17:31
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.

Support databases protected with ACL (user + password authentication)

3 participants