TNO-S3 / WuppieFuzz

A coverage-guided REST API fuzzer developed on top of LibAFL
Apache License 2.0
93 stars 4 forks source link

Resolve references (or ignore) text/plain body instead of todo!() #40

Closed grebnetiew closed 3 weeks ago

grebnetiew commented 3 weeks ago

Addresses #38 (hopefully)

github-actions[bot] commented 3 weeks ago

Sigrid maintainability feedback

✅ You wrote maintainable code and achieved your Sigrid objective of 3.5 stars

Sigrid compared your code against the baseline of 2024-10-30.

👍 What went well?

You fixed or improved 0 refactoring candidates.

👎 What could be better?

Unfortunately, 1 refactoring candidates were introduced or got worse.

Risk System property Location
🟠 Module Coupling
(Worsened)
WuppieFuzz/src/input/mod.rs

📚 Remaining technical debt

6 refactoring candidates didn't get better or worse, but are still present in the code you touched.

Risk System property Location
🟠 Unit Size
(Unchanged)
WuppieFuzz/src/input/mod.rs
fix_broken_references(R)
🟡 Unit Size
(Unchanged)
WuppieFuzz/src/input/mod.rs
Body.build(OpenAPI,Operation,Option)
🟡 Unit Size
(Unchanged)
WuppieFuzz/src/input/mod.rs
Input.generate_name(Option)
🟡 Unit Size
(Unchanged)
WuppieFuzz/src/input/mod.rs
fix_input_parameters(S,usize,OpenApiRequest)
🟡 Unit Interfacing
(Unchanged)
WuppieFuzz/src/input/mod.rs
Body.build(OpenAPI,Operation,Option)
🟡 Unit Interfacing
(Unchanged)
WuppieFuzz/src/input/mod.rs
fix_input_parameters(S,usize,OpenApiRequest)

Sigrid ratings

System property System on 2024-10-30 Before changes New/changed code
Volume 5.4 N/A N/A
Duplication 3.9 5.5 5.5
Unit Size 2.1 1.0 1.0
Unit Complexity 2.9 5.5 5.5
Unit Interfacing 2.0 1.6 1.6
Module Coupling 3.3 N/A N/A
Component Independence 5.4 N/A N/A
Component Entanglement N/A N/A N/A
Maintainability 3.6 3.9 3.9

View this system in Sigrid


Did you find this feedback helpful?

We would like to know your thoughts to make Sigrid better. Your username will remain confidential throughout the process.

ThomasTNO commented 3 weeks ago

LGTM, can you add a changlog.md entry?

grebnetiew commented 3 weeks ago

LGTM, can you add a changlog.md entry?

Like so?