Skip to content

fix: index-out-of-bounds event parser panic + regression test#2353

Closed
sergeytimoshin wants to merge 1 commit intomainfrom
sergey/event-parser-fix
Closed

fix: index-out-of-bounds event parser panic + regression test#2353
sergeytimoshin wants to merge 1 commit intomainfrom
sergey/event-parser-fix

Conversation

@sergeytimoshin
Copy link
Contributor

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 18, 2026

Important

Review skipped

Ignore keyword(s) in the title.

⛔ Ignored keywords (5)
  • wip
  • draft
  • temp
  • test
  • experimental

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: e3ee8c70-8a14-441e-a4db-ff491094d70b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sergey/event-parser-fix
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

You can disable the changed files summary in the walkthrough.

Disable the reviews.changed_files_summary setting to disable the changed files summary in the walkthrough.

i-am-bert added a commit to helius-labs/photon that referenced this pull request Mar 18, 2026
Bumps light-event from 0.21 to 0.24, which will include the fix for the
index-out-of-bounds panic in create_nullifier_queue_indices() that halted
both pitt and fra ingestors at slot 407265370.

Upstream fix: Lightprotocol/light-protocol#2353

Note: This PR should be merged and deployed once light-event 0.24.0 is
published to crates.io. Until then, cargo build will fail to resolve the
dependency.
i-am-bert added a commit to helius-labs/photon that referenced this pull request Mar 18, 2026
Bumps light-event from 0.21 to 0.23.1, which includes the fix for the
index-out-of-bounds panic in create_nullifier_queue_indices() that halted
both pitt and fra ingestors at slot 407265370.

Upstream fix: Lightprotocol/light-protocol#2353
i-am-bert added a commit to helius-labs/photon that referenced this pull request Mar 18, 2026
Bumps all light-* dependencies to compatible versions:
- light-event: 0.21 -> 0.23.1 (includes OOB panic fix)
- light-concurrent-merkle-tree: 4 -> 5
- light-batched-merkle-tree: 0.10 -> 0.11
- light-merkle-tree-metadata: 0.10 -> 0.11
- light-compressed-account: 0.10.1 -> 0.11
- light-indexed-merkle-tree: 4 -> 5
- light-token-interface: 0.4 -> 0.5
- light-sdk-types: 0.20 -> 0.23

Fixes the index-out-of-bounds panic in create_nullifier_queue_indices()
that halted both pitt and fra ingestors at slot 407265370.

Upstream fix: Lightprotocol/light-protocol#2353
cargo check passes.
hetdagli234 pushed a commit to helius-labs/photon that referenced this pull request Mar 18, 2026
Bumps all light-* dependencies to compatible versions:
- light-event: 0.21 -> 0.23.1 (includes OOB panic fix)
- light-concurrent-merkle-tree: 4 -> 5
- light-batched-merkle-tree: 0.10 -> 0.11
- light-merkle-tree-metadata: 0.10 -> 0.11
- light-compressed-account: 0.10.1 -> 0.11
- light-indexed-merkle-tree: 4 -> 5
- light-token-interface: 0.4 -> 0.5
- light-sdk-types: 0.20 -> 0.23

Fixes the index-out-of-bounds panic in create_nullifier_queue_indices()
that halted both pitt and fra ingestors at slot 407265370.

Upstream fix: Lightprotocol/light-protocol#2353
cargo check passes.

Co-authored-by: i-am-bert <i-am-bert@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant