Skip to content
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

add getTransactions endpoint #110

Closed
Tracked by #115
mollykarcher opened this issue Mar 21, 2024 · 3 comments
Closed
Tracked by #115

add getTransactions endpoint #110

mollykarcher opened this issue Mar 21, 2024 · 3 comments
Assignees

Comments

@mollykarcher
Copy link
Contributor

mollykarcher commented Mar 21, 2024

What problem does your feature solve?

Horizon will cease to serve txmeta sometime in Q2 (see discord discussion) due to data storage size concerns of Soroban transactions. The ecosystem needs an alternative to this, and we want that alternative to be the RPC.

What would you like to see?

Users need to be able to page through all transactions available in RPC's retention window. See this issue for some prior discussion/context on why we've chosen not to implement an actual stream/websocket/sse implementation here.

The getTransactions endpoint should be able to support pagination and should provide all metadata about the transaction that is currently available via getTransaction.

Scope for this will be limited to simple pagination with no filters, and we will work/propose additional filters on this as separate issues. Pagination should follow the format/user experience of the getEvents endpoint, except see (see #118) for changes that we want to make to pagination and follow the decision there.

Request format:

{
  "jsonrpc": "2.0",
  "id": <id>,
  "method": "getTransactions",
  "params": {
    "startLedger": <startLedger>,
    "endLedger": <endLedger
    "pagination": {
      "limit": <limit>,
      "cursor": <cursor>
    }
  }
}

Response format

{
  "jsonrpc": "2.0",
  "id": <id>,
  "result": {
    "transactions": [
        {
            "status": <status>,
            "applicationOrder": <order>,
            "envelopeXdr": <envelopeXdr>,
            "resultXdr": <resultXdr>,
            "resultMetaXdr": <resultMetaXdr>,
            "ledger": <ledger>,
            "createdAt": <createdAt>
        }
    ],
    "latestLedger": <latestLedger>,
    "latestLedgerCloseTime": <closeTime>,
    "oldestLedger": <oldestLedger>,
    "oldestLedgerCloseTime": <oldestCloseTime>,
    "cursor": <cursor / pagingToken>
  }
}

Tasks

No tasks being tracked yet.
@mollykarcher
Copy link
Contributor Author

@psheth9 @aditya1702 I added an example request/response to the description here, per our conversation yesterday. Let me know what you think

@Shaptic
Copy link
Contributor

Shaptic commented Apr 5, 2024

    "latestLedger": <latestLedger>,
    "latestLedgerCloseTime": <closeTime>,
    "oldestLedger": <oldestLedger>,
    "oldestLedgerCloseTime": <oldestCloseTime>,

I know this isn't hard to provide, but why do we need this?

Also, what is <ledger> in each transaction? Is is the LedgerCloseMeta, just the header, or...?

This also needs pagination information in the response, e.g. a "cursor" field for the next set of ledgers.

@mollykarcher
Copy link
Contributor Author

I know this isn't hard to provide, but why do we need this?

I just included this because it already exists in getTransaction, but truthfully I'm not sure I know the historical reasoning so worth it to decide if we want it. If I had to guess, it's probably because it could make it easier to reason about the response from getTransaction in the case where you receive a 404. If you get a 404, it could mean 1) your transaction is "pending" and hasn't been included in a ledger yet 2) your transaction is older than this RPC instance's history and therefore you can't fetch it 3) that transaction actually doesn't exist. So oldestLedger fields could help you discern between cases 1/2.

Also, what is in each transaction? Is is the LedgerCloseMeta, just the header, or...?

The ledger sequence number. Again, mimicking existing getTransaction fields

This also needs pagination information in the response, e.g. a "cursor" field for the next set of ledgers.

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

4 participants