DutchEllie
/
pepebot
1
0
Fork 0
Code Issues 2 Pull Requests Packages Projects Releases Wiki Activity

fixed bug in removing actions

This commit is contained in:
DutchEllie 2021-05-31 11:35:46 +02:00
parent 4460e988fd
commit 5228a9be38
1 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
limiter/limiter.go
View File

@ -28,11 +28,11 @@ func (l *Limiter) LogInteraction(userid string, action string) {
Returns an error if the amount of log entries exceeds the ratelimit */ Returns an error if the amount of log entries exceeds the ratelimit */
func (l *Limiter) CheckAllowed(userid string) error { func (l *Limiter) CheckAllowed(userid string) error {
counter := 0 counter := 0
expiredEntries := make([]int, 0) expiredEntries := make([]*Action, 0)
for i := 0; i < len(l.Logs[userid]); i++ { for i := 0; i < len(l.Logs[userid]); i++ {
/* If the timestamp plus the timelimit is happened before "Now" */ /* If the timestamp plus the timelimit is happened before "Now" */
if l.Logs[userid][i].Timestamp.Add(l.TimeLimit).Before(time.Now()) { if l.Logs[userid][i].Timestamp.Add(l.TimeLimit).Before(time.Now()) {
expiredEntries = append(expiredEntries, i) expiredEntries = append(expiredEntries, l.Logs[userid][i])
continue continue
} else { } else {
counter++ counter++
@ -52,7 +52,14 @@ func (l *Limiter) CheckAllowed(userid string) error {
} }
} }
func (l *Limiter) removeAction(userid string, i int) { func (l *Limiter) removeAction(userid string, a *Action) {
l.Logs[userid][i] = l.Logs[userid][len(l.Logs[userid])-1] index := 0
for i := 0; i < len(l.Logs[userid]); i++ {
if l.Logs[userid][i] == a {
index = i
break
}
}
l.Logs[userid][index] = l.Logs[userid][len(l.Logs[userid])-1]
l.Logs[userid] = l.Logs[userid][:len(l.Logs[userid])-1] l.Logs[userid] = l.Logs[userid][:len(l.Logs[userid])-1]
} }