Skip to content

testing: link to fuzzing guide and more details on runtime #51032

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.

Sign up for GitHub

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

Jump to bottom
Closed
ronaldpetty opened this issue Feb 6, 2022 · 4 comments
Closed

testing: link to fuzzing guide and more details on runtime #51032

ronaldpetty opened this issue Feb 6, 2022 · 4 comments
Labels
Documentation Issues describing a change to documentation. fuzz Issues related to native fuzzing support NeedsFix The path to resolution is known, but the work has not been done.
Milestone
Backlog

Comments

@ronaldpetty
Copy link 

$ go version
go version go1.18beta2 darwin/amd64

The go doc testing and this page https://pkg.go.dev/testing@master#hdr-Fuzzing still are pointing to #48255. They mention:

TODO(#48255): write and link to documentation that will be helpful to users who are unfamiliar with fuzzing.

Some details such as:

  • why does it run long (∞) and how to stop
  • maybe example quickly shows it 'finding' evil params (the current one ran for 30+ minutes and nothing found)
    • just to show what happens when it does, else you just see endless processing

I was asked to create new ticket (versus comment in #48255).

Looking forward to using it, and thanks for the hard work.
@ianlancetaylor
Copy link
Member

CC @golang/fuzzing

@ianlancetaylor ianlancetaylor added the Documentation Issues describing a change to documentation. label Feb 6, 2022
@seankhliao seankhliao changed the title Guide for Fuzzing testing: link to fuzzing guide and more details on runtime Feb 6, 2022
@katiehockman
Copy link
Contributor

https://go-review.googlesource.com/c/go/+/383415 fixes the TODO in the testing package 👍

@katiehockman katiehockman added the fuzz Issues related to native fuzzing support label Feb 7, 2022
@katiehockman katiehockman added this to the Backlog milestone Feb 7, 2022
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/383454 mentions this issue: _content/doc/fuzz: describe the nature of a failure

@ronaldpetty
Copy link
Author

Thank you @katiehockman . I have solved all my questions after seeing that link (and related ones). Cool feature.

@cherrymui cherrymui added the NeedsFix The path to resolution is known, but the work has not been done. label Feb 8, 2022
gopherbot pushed a commit to golang/website that referenced this issue Feb 8, 2022
@katiehockman
_content/doc/fuzz: describe the nature of a failure
b7142a0 
Updates golang/go#51032

Change-Id: Iec7965f8d2823e7918967e0fae8425e0b1cc615e
Reviewed-on: https://go-review.googlesource.com/c/website/+/383454
Run-TryBot: Katie Hockman <katie@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Roland Shoemaker <bracewell@google.com>
Trust: DO NOT USE <katiehockman@google.com>
Run-TryBot: DO NOT USE <katiehockman@google.com>
MK825 added a commit to MK825/website that referenced this issue Oct 18, 2022
@MK825
_content/doc/fuzz: describe the nature of a failure
c3b2b4c 
Updates golang/go#51032

Change-Id: Iec7965f8d2823e7918967e0fae8425e0b1cc615e
Reviewed-on: https://go-review.googlesource.com/c/website/+/383454
Run-TryBot: Katie Hockman <katie@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Roland Shoemaker <bracewell@google.com>
Trust: DO NOT USE <katiehockman@google.com>
Run-TryBot: DO NOT USE <katiehockman@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation Issues describing a change to documentation. fuzz Issues related to native fuzzing support NeedsFix The path to resolution is known, but the work has not been done.
Projects
Go Security
Status: Done
Milestone
Backlog
Development

No branches or pull requests
6 participants
@ronaldpetty @ianlancetaylor @gopherbot @seankhliao @cherrymui @katiehockman