Remove Bug type - Githubissues

FuelLabs / fuel-vm

Fuel v2 interpreter in Rust

Other

354 stars 86 forks source link

Remove Bug type #752

Closed Dentosal closed 4 months ago

Dentosal commented 4 months ago

Removes Bug and BugVariant types, opting to expect or unreachable! instead, as suggested in audit reports. Since we already catch the rust-level panics in fuel-core, we should move the reporting there. This way, we include all crashes and not just those where we had wrapped it in Bug.

Checklist

[ ] Breaking changes are clearly marked as such in the PR description and changelog

Before requesting review

[ ] I have reviewed the code myself
[ ] I have created follow-up issues caused by this PR and linked them here

xgreenx commented 4 months ago

As we discussed internally: let's stick with Bug for now to not change the behavior of the current code