Skip to content

getledgerentry endpoint for RPC#4721

Merged
SirTyson merged 2 commits intostellar:masterfrom
SirTyson:rpc-endpoint
May 19, 2025
Merged

getledgerentry endpoint for RPC#4721
SirTyson merged 2 commits intostellar:masterfrom
SirTyson:rpc-endpoint

Conversation

@SirTyson
Copy link
Contributor

Description

This PR adds an HTTP endpoint that can query ledger state and provide an entry's archival status. This can be used by RPC for preflighting TXs without needing to maintain a redundant ledger state database. The endpoint API can be found in docs/software/commands.md.

Checklist

  • Reviewed the contributing document
  • Rebased on top of master (no merge commits)
  • Ran clang-format v8.0.0 (via make format or the Visual Studio extension)
  • Compiles
  • Ran all tests
  • If change impacts performance, include supporting evidence per the performance document

Copy link
Member

@leighmcculloch leighmcculloch left a comment

Choose a reason for hiding this comment

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

👏🏻 Reviewing just the command doc.

@SirTyson SirTyson requested a review from graydon May 12, 2025 22:25
@SirTyson
Copy link
Contributor Author

I should have addressed all y'alls comments and unified the interfaces.

dmkozh
dmkozh previously approved these changes May 14, 2025
graydon
graydon previously approved these changes May 15, 2025
Copy link
Contributor

@graydon graydon left a comment

Choose a reason for hiding this comment

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

LGTM

@SirTyson SirTyson enabled auto-merge May 15, 2025 18:40
Copy link

@Shaptic Shaptic left a comment

Choose a reason for hiding this comment

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

Reviewed docs, not code, but API looks good!

@SirTyson SirTyson dismissed stale reviews from graydon and dmkozh via 94ff674 May 18, 2025 00:00
@SirTyson SirTyson added this pull request to the merge queue May 19, 2025
Merged via the queue into stellar:master with commit 88cd5b9 May 19, 2025
13 checks passed
@SirTyson SirTyson deleted the rpc-endpoint branch May 19, 2025 17:10
Shaptic added a commit to stellar/go-stellar-sdk that referenced this pull request May 21, 2025
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.

5 participants