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 transaction indexing flag to rpc #1005

Open
dimiandre opened this issue Nov 14, 2022 · 2 comments
Open

Add transaction indexing flag to rpc #1005

dimiandre opened this issue Nov 14, 2022 · 2 comments

Comments

@dimiandre
Copy link
Member

It's very annoying when RPC doesn't have transaction indexing enabled, would be useful to know which RPC endpoints provide it and which not

Would also be cool to have a proxy only for indexing-enabled ones with cosmos.directory

(node:20819) UnhandledPromiseRejectionWarning: Error: {"code":-32603,"message":"Internal error","data":"transaction indexing is disabled"}

@tombeynon
Copy link
Contributor

I'm going to add a feature to cosmos.directory very soon that will monitor whether nodes have indexing enabled, and provide this data via the status.cosmos.directory endpoints.

I also plan to accept a header for the proxies to ensure the request is sent to an indexed node, something like x-cosmos-index-enabled so it's inline with the similar x-cosmos-block-height header that the SDK REST API already accepts. Any suggestions for naming this header would be helpful, like if we should avoid the x-cosmos namespace for example.

I plan to have this implemented in January if at all possible, will post here with any updates.

@JeremyParish69
Copy link
Collaborator

JeremyParish69 commented Dec 19, 2022

Would a simple "indexing" boolean be clear and sufficient?:


...
"endpoint": {
      "type": "object",
      "required": [
        "address"
      ],
      "properties": {
        "address": {
          "type": "string"
        },
        "provider": {
          "type": "string"
        },
        "archive": {
          "type": "boolean",
          "default": false
        },
        "indexing": {
          "type": "boolean",
          "default": false
        }
      }
    },
...

or do we need more, like transaction_indexing ?

@cosmos cosmos deleted a comment from thallyssonnascimento Dec 16, 2023
@cosmos cosmos deleted a comment from thallyssonnascimento Dec 16, 2023
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

No branches or pull requests

4 participants
@dimiandre @tombeynon @JeremyParish69 and others