Skip to content

Conversation

@EugeneDrpc
Copy link
Contributor

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request fixes null pointer exceptions and improves 404 error detection across beacon chain lower bound detectors.

Changes:

  • Fixed NPE in blob detector when handling null or empty data nodes
  • Improved 404 error code detection to handle both numeric (404) and text ("404") code formats across all detectors
  • Enhanced null-safety by storing node references before accessing their properties

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
BeaconChainLowerBoundStateDetector.kt Updated 404 detection logic to handle both numeric and text code formats, preventing NPE when code is numeric
BeaconChainLowerBoundEpochDetector.kt Updated 404 detection logic to handle both numeric and text code formats, preventing NPE when code is numeric
BeaconChainLowerBoundBlockDetector.kt Updated 404 detection logic to handle both numeric and text code formats, preventing NPE when code is numeric
BeaconChainLowerBoundBlobDetector.kt Fixed NPE when data node is null and improved 404 detection to handle both numeric and text code formats

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@EugeneDrpc EugeneDrpc merged commit 01c51a6 into master Jan 20, 2026
1 check passed
@EugeneDrpc EugeneDrpc deleted the fix-blob-detector branch January 20, 2026 10:55
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.

4 participants