Duplicate false positive causes panic

[quackduck]

https://github.com/TwiN/go-away/blob/master/goaway.go#L178

runtime error: slice bounds out of range [457:451]

[TwiN]

@quackduck With what string did this happen, so that I can replicate it on my end?

[quackduck]

I couldn't find the exact string but this has the same effect:

panic: runtime error: slice bounds out of range [392:380]

package main

import (
    "github.com/TwiN/go-away"
    "fmt"
)

var text = `pi@raspberrypi:~/Documents/devzat-bot $ yarn
yarn install v1.22.18
warning package.json: No license field
warning No license field
[1/4] Resolving packages...
success Already up-to-date.
Done in 0.61s.
pi@raspberrypi:~/Documents/devzat-bot $ yarn start
yarn run v1.22.18
warning package.json: No license field
error Command "start" not found.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
pi@raspberrypi:~/Documents/devzat-bot $`

func main() {
    fmt.Println(goaway.Censor(text))
}

[TwiN]

Thanks! I was able to reproduce it using the following string:

documentdocument

Basically, it happens when there's two false positives.

This bug appears to have been introduced by #27, which was released as part of v1.6.1. In the meantime, you can use a version older than that.

[quackduck]

Epic, thanks!

[TwiN]

Released in v1.6.3.
Also opened #31 because I discovered another but while working on this.