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

[ledger_entry with invalid combination of parameters return internal error] (Version: [1.11.0-rc2]]) #4550

Closed
mounikakun opened this issue May 30, 2023 · 0 comments · Fixed by #4630
Labels

Comments

@mounikakun
Copy link
Collaborator

Issue Description

Steps to Reproduce

{ "method": "ledger_entry", "params": [ { "check": { "owner": "rhigTLJJyXXSRUyRCQtqi1NoAZZzZnS4KU", "sub_index": 0 }, "ledger_index": "validated" } ] }

Expected Result

invalidParams

Actual Result

{ "result": { "error": "internal", "error_code": 73, "error_message": "Internal error.", "request": { "check": { "owner": "rhigTLJJyXXSRUyRCQtqi1NoAZZzZnS4KU", "sub_index": 0 }, "command": "ledger_entry", "ledger_index": "validated" }, "status": "error" } }

Environment

Supporting Files

@intelliot intelliot linked a pull request Sep 7, 2023 that will close this issue
intelliot pushed a commit that referenced this issue Sep 8, 2023
- Verify "check", used to retrieve a Check object, is a string.
- Verify "nft_page", used to retrieve an NFT Page, is a string.
- Verify "index", used to retrieve any type of ledger object by its
  unique ID, is a string.
- Verify "directory", used to retrieve a DirectoryNode, is a string or
  an object.

This change only impacts api_version 2 since it is a breaking change.

https://xrpl.org/ledger_entry.html

Fix #4550
gregtatcam pushed a commit to gregtatcam/rippled that referenced this issue Sep 8, 2023
)

- Verify "check", used to retrieve a Check object, is a string.
- Verify "nft_page", used to retrieve an NFT Page, is a string.
- Verify "index", used to retrieve any type of ledger object by its
  unique ID, is a string.
- Verify "directory", used to retrieve a DirectoryNode, is a string or
  an object.

This change only impacts api_version 2 since it is a breaking change.

https://xrpl.org/ledger_entry.html

Fix XRPLF#4550
ckeshava pushed a commit to ckeshava/rippled that referenced this issue Sep 22, 2023
)

- Verify "check", used to retrieve a Check object, is a string.
- Verify "nft_page", used to retrieve an NFT Page, is a string.
- Verify "index", used to retrieve any type of ledger object by its
  unique ID, is a string.
- Verify "directory", used to retrieve a DirectoryNode, is a string or
  an object.

This change only impacts api_version 2 since it is a breaking change.

https://xrpl.org/ledger_entry.html

Fix XRPLF#4550
ckeshava pushed a commit to ckeshava/rippled that referenced this issue Sep 25, 2023
)

- Verify "check", used to retrieve a Check object, is a string.
- Verify "nft_page", used to retrieve an NFT Page, is a string.
- Verify "index", used to retrieve any type of ledger object by its
  unique ID, is a string.
- Verify "directory", used to retrieve a DirectoryNode, is a string or
  an object.

This change only impacts api_version 2 since it is a breaking change.

https://xrpl.org/ledger_entry.html

Fix XRPLF#4550
sophiax851 pushed a commit to sophiax851/rippled that referenced this issue Jun 12, 2024
)

- Verify "check", used to retrieve a Check object, is a string.
- Verify "nft_page", used to retrieve an NFT Page, is a string.
- Verify "index", used to retrieve any type of ledger object by its
  unique ID, is a string.
- Verify "directory", used to retrieve a DirectoryNode, is a string or
  an object.

This change only impacts api_version 2 since it is a breaking change.

https://xrpl.org/ledger_entry.html

Fix XRPLF#4550
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants