Page MenuHomePhabricator

[Chronik] Add `should_ignore` to `empp::ParseError`
ClosedPublic

Authored by tobias_ruck on Dec 16 2023, 13:57.

Details

Reviewers
Fabien
PiRK
Group Reviewers
Restricted Project
Commits
rABC9877fd6af1b2: [Chronik] Add `should_ignore` to `empp::ParseError`
Summary

When reporting errors from the indexer, it isn't helpful to know that e.g. a P2PKH-only tx isn't eMPP compliant and Chronik shouldn't refuse to broadcast them.

However, if a script starts with OP_RETURN OP_RESERVED, it's pretty clear that the intent is to use eMPP, and therefore we report the error and refuse broadcast.

Also, refactor some small things like using Opcode instead of Op in ParseError.

Test Plan

cargo test -p bitcoinsuite-slp

Diff Detail

Repository
rABC Bitcoin ABC
Branch
chronik-empp-should-ignore
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 25983
Build 51540: Build Diffchronik-client-integration-tests · build-chronik
Build 51539: arc lint + arc unit