Skip to content

Conversation

fzipi
Copy link
Member

@fzipi fzipi commented Jul 28, 2025

what

  • add raw file corpus support

why

  • easy test without leipzig

refs

Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
@fzipi fzipi requested a review from theseion July 28, 2025 11:47
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about making leipzig and raw subcommands of corpus. They can inherit all the stuff from the corpus command (which wouldn't have any functionality itself) and implementation would be very concise for each of them (and new types).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this would be then go-ftw quantitative corpus raw .... for example?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes

Comment on lines +37 to +41
s.tempDir, err = os.MkdirTemp("", "raw_corpus_test")
s.Require().NoError(err)

// Create a test file with sample payloads
s.tempFile = filepath.Join(s.tempDir, "test_corpus.txt")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
s.tempDir, err = os.MkdirTemp("", "raw_corpus_test")
s.Require().NoError(err)
// Create a test file with sample payloads
s.tempFile = filepath.Join(s.tempDir, "test_corpus.txt")
// Create a test file with sample payloads
s.tempFile = filepath.Join(s.T().TempDir(), "test_corpus.txt")

suite.Suite
corpus corpus.Corpus
tempFile string
tempDir string
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
tempDir string

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature: add option to use custom corpus file
2 participants