Mixed-case transaction hashes are silently excluded from the allowlist

Open Beginner friendly
#193 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
82/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
typescript
Domain
backend

Research direction

Start by reading validator-ejector/src/services/config/service.ts:73-80 and validator-ejector/src/services/exit-logs/verifier.ts:365-375. Trace how SUBMIT_TX_HASH_ALLOWLIST is parsed and compared, then verify that an exact mixed-case transaction hash is accepted consistently with its lowercase form.

Written by the indexing model from the issue text.

Description

Locations

Summary

SUBMIT_TX_HASH_ALLOWLIST preserves the operator’s input casing, but verification lowercases only the observed transaction hash before a case-sensitive comparison. A semantically exact mixed-case hash is therefore treated as unauthorized.

Root cause

Configuration parsing accepts transaction hashes as unchanged strings. verifySubmitExitRequestsDataTransaction subsequently compares transactionHash.toLowerCase() directly against that unnormalized array.

Dominant language
TypeScript
Stars
16
Forks
19
Avg merge
1d 20h
Merged PRs (30d)
7

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from lidofinance/validator-ejector

All issues in lidofinance/validator-ejector

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.