jazzband / django-defender

A simple super fast django reusable app that blocks people from brute forcing login attempts
Apache License 2.0
1.03k stars 144 forks source link

add username option to redis config #218

Closed erdos4d closed 2 years ago

erdos4d commented 2 years ago

When setting DEFENDER_REDIS_URL, add capability to use non-default username.

codecov[bot] commented 2 years ago

Codecov Report

Merging #218 (eab8a1e) into master (0b96c53) will increase coverage by 0.03%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #218      +/-   ##
==========================================
+ Coverage   92.46%   92.50%   +0.03%     
==========================================
  Files          13       13              
  Lines         398      400       +2     
==========================================
+ Hits          368      370       +2     
  Misses         30       30              
Impacted Files Coverage Δ
defender/connection.py 96.77% <100.00%> (+0.22%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0b96c53...eab8a1e. Read the comment docs.

aleksihakli commented 2 years ago

LGTM, thank you for the PR :) @kencochrane is the person who can make a new version release to PyPI for django-defender.

kencochrane commented 2 years ago

It should be released to pypi. Thanks!