Skip to content

fix: allowed callTracer to return empty array if no call actions found #3768

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

quiet-node
Copy link
Contributor

Description:
This PR allows callTracer to return an empty array if no call actions are found, effectively preventing the deconstruction of an undefined object.

Related issue(s):

Fixes #3751

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@quiet-node quiet-node added this to the 0.69.0 milestone May 15, 2025
@quiet-node quiet-node self-assigned this May 15, 2025
@quiet-node quiet-node added the bug Something isn't working label May 15, 2025
@quiet-node quiet-node requested review from a team as code owners May 15, 2025 19:00
@quiet-node quiet-node linked an issue May 15, 2025 that may be closed by this pull request
@quiet-node quiet-node requested a review from simzzz May 15, 2025 19:00
@lfdt-bot
Copy link

lfdt-bot commented May 15, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

Copy link

github-actions bot commented May 15, 2025

Test Results

 22 files  + 1  271 suites   - 2   36m 52s ⏱️ + 3m 24s
646 tests + 1  639 ✅ ± 0  5 💤 ±0  2 ❌ +1 
761 runs  +60  752 ✅ +57  7 💤 +2  2 ❌ +1 

For more details on these failures, see this check.

Results for commit 09ae045. ± Comparison against base commit fca86cc.

♻️ This comment has been updated with latest results.

acuarica
acuarica previously approved these changes May 15, 2025
Copy link

@Ferparishuertas Ferparishuertas left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
@quiet-node quiet-node dismissed stale reviews from Ferparishuertas and acuarica via 09ae045 May 16, 2025 16:57
@quiet-node quiet-node force-pushed the 3751-debug_tracetransaction-fails-with-some-specific-tx-hash branch from ecea0d7 to 09ae045 Compare May 16, 2025 16:57
Copy link
Contributor

@acuarica acuarica left a comment

Choose a reason for hiding this comment

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

Approved, but left a comment.

Copy link

@Ferparishuertas Ferparishuertas left a comment

Choose a reason for hiding this comment

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

LGTM

@quiet-node quiet-node merged commit 5569a35 into main May 16, 2025
45 of 46 checks passed
@quiet-node quiet-node deleted the 3751-debug_tracetransaction-fails-with-some-specific-tx-hash branch May 16, 2025 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

debug_traceTransaction fails with some specific tx hash
5 participants