Skip to content

Commit

Permalink
main: deprecate the old rate limiters
Browse files Browse the repository at this point in the history
That is, all except the distributed one. As with the API Healthcheck
endpoints, we are not removing these yet - just adding the warning for
now.

Fixes #797.
  • Loading branch information
mvdan committed Oct 10, 2017
1 parent 8e7743a commit 12e09bd
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions main.go
Original file line number Diff line number Diff line change
Expand Up @@ -1162,9 +1162,11 @@ func handleDashboardRegistration() {

func startDRL() {
switch {
case config.Global.ManagementNode,
config.Global.EnableSentinelRateLImiter,
case config.Global.ManagementNode:
return
case config.Global.EnableSentinelRateLImiter,
config.Global.EnableRedisRollingLimiter:
log.Warning("The old, non-distributed rate limiter is deprecated and we no longer recommend its use.")
return
}
log.WithFields(logrus.Fields{
Expand Down

0 comments on commit 12e09bd

Please sign in to comment.