-
Notifications
You must be signed in to change notification settings - Fork 212
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
🔒 Fuzzing setup and fixes #754
Conversation
Why do you not allow fuzzing on Windows? |
@moorereason the Go fuzz runner seems to be failing for some internal reason on windows. Figured I’d block it until I can figure out what’s wrong there. |
I tested the Windows fuzzing piece and opened golang/go#52268. The work-around is some form of these Git settings:
|
Also, please keep this |
Cool! Thank you for looking into this. I've added branch protection on this one to make sure it does not go away. Creating a new PR to get those fixes in. I'll try to play with |
Confirmed that adding an entry to |
@moorereason since golang/go#52268 has been closed, mind if I close this one and delete the branch too? |
Agreed |
Finding and fixing bugs in the encoder with Go 1.18 fuzzing.