Repository navigation

fix(login): rate limiter shouldn't count successful logins - #3141

Merged

repo-ranger[bot] merged 14 commits into

mainfrom

jsjoeio/fix-login-rate-limiter

Apr 19, 2021

Merged

fix(login): rate limiter shouldn't count successful logins#3141
repo-ranger[bot] merged 14 commits into
mainfrom
jsjoeio/fix-login-rate-limiter

Commits on Apr 19, 2021
- fix: update comment and export rateLimiter
  jsjoeio
  committed
  4683d8a
- feat(testing): add tests for RateLimiter
  jsjoeio
  committed
  58e17c5
- refactor(testing): combine loginPage with login
  jsjoeio
  committed
  ebbabc6
- feat(testing): add e2e tests for password
  jsjoeio
  committed
  faaa0a9
- feat: increase timeout for playwright tests
  jsjoeio
  committed
  83cfbf8
- refactor(login): move rate limiter after successful login
  jsjoeio
  committed
  0852107
- feat(testing): add test for rate limiter
  jsjoeio
  committed
  1e6f4f2
- refactor: change config to save all e2e videos
  jsjoeio
  committed
  a8719e1
- refactor: update rateLimiter to check try
  jsjoeio
  committed
  d8e4505
- feat: add test for limiter.canTry()
  jsjoeio
  committed
  7928dc2
- refactor: change limiter.Try() to .removeToken()
  jsjoeio
  committed
  a3f18d6
- refactor: check errorMessage in login e2e test
  jsjoeio
  committed
  958f012
- fix: update logic for removing token from limiter
  jsjoeio
  committed
  7a50421
- refactor: rateLimiter.canTry logic to check >= 1
  jsjoeio
  committed
  f80d5c3