-
-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bug: panic, when using utf-8 #47
Labels
bug
Something isn't working
good first issue
Good for newcomers
help wanted
Extra attention is needed
Comments
Looks like a duplicate of #43 -- which likely has to do with the fact that UTF-8 characters can be more than one byte. |
TwiN
added
good first issue
Good for newcomers
help wanted
Extra attention is needed
labels
Feb 18, 2023
I attempted a fix on this here: #64 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
Something isn't working
good first issue
Good for newcomers
help wanted
Extra attention is needed
Describe the bug
use the test:
What do you see?
panic at:
github.com/TwiN/go-away.(*ProfanityDetector).Censor(0xc000082f08, {0xf8f7c7, 0x3c})
.../github.com/TwiN/go-away/goaway.go:193 +0x7a5
What do you expect to see?
UTF-8 filtered as well as ASCII
List the steps that must be taken to reproduce this issue
use the test from the description
Version
v1.6.9
Additional information
No response
The text was updated successfully, but these errors were encountered: