hiero-ledger / hiero-ledger/hiero-consensus-node

Add due diligence check

Open
#17,451 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
406
Forks
226
Avg merge
3d 4h
Merged PRs (30d)
210

Description

### Background

Create a check in the preHandle workflow that creates a due diligence error if a transaction does not follow the new normalized format.

### Acceptance Criteria

1. A transaction that uses `bodyBytes` to store the `TransactionBody` fails with a due diligence error.
2. A transaction that uses `signedTransactionBytes` also fails with a due diligence error.

### Dependencies

1. Prerequisite: #17448
2. Prerequisite: #17452

### Definition of Ready (DoR) Checklist

- [ ] Clear acceptance criteria
- [ ] Clear and detailed description
- [ ] Dependencies identified
- [ ] Links to documentation
- [ ] Should be completable in 2-3 Days
- [ ] Initial draft of Low-level design document
- [ ] At least high level test plan
- [ ] Groomed/Estimated

### Definition of Done (DoD) Checklist

- [ ] Acceptance Criteria complete
- [ ] No Codacy issues greater than minor (in new code)
- [ ] JavaDocs updated/created
- [ ] Code commented
- [ ] Unit tests created/updated
- [ ] 80% test code coverage (in new code)
- [ ] Happy Path and major negative cases in HAPI tests as applicable

Contributor guide

Open the contributing guide

Research direction

Review the preHandle workflow and prerequisites #17448 and #17452 first; the issue does not name files or entry points beyond that workflow. Done means transactions using bodyBytes or signedTransactionBytes produce a due diligence error, with unit tests and applicable HAPI tests updated.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.