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.

Sign up for GitHub

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

Jump to bottom

“gateway_balances” with unfunded account do not throw an error (Version: [1.9.2]) #4290

Closed
mounikakun opened this issue Aug 30, 2022 · 0 comments · Fixed by #4618
Closed

“gateway_balances” with unfunded account do not throw an error (Version: [1.9.2]) #4290

mounikakun opened this issue Aug 30, 2022 · 0 comments · Fixed by #4618
Labels
API Change Will Need Documentation

Comments

@mounikakun
Copy link
Collaborator

Issue Description

If the [Address]specified in the account field of the request does not correspond to an account in the ledger then response should result in actNotFound error.

Steps to Reproduce

{ "method": "gateway_balances", "params": [ { "account": "rNGQLojaFxTYphuQUJ24QUhyGBUMMbqBMx", "ledger_index": "validated" } ] }

Expected Result

actNotFound

Actual Result

{ "result": { "account": "rNGQLojaFxTYphuQUJ24QUhyGBUMMbqBMx", "ledger_hash": "CE0AFDF8B84168937941DAEF469AF1D03BE4F8C5532DA77652A18D707FD86981", "ledger_index": 21321107, "status": "success", "validated": true } }
@PeterChen13579 PeterChen13579 mentioned this issue Jun 20, 2023
Closed
1 task
@PeterChen13579 PeterChen13579 mentioned this issue Jul 12, 2023
Merged
@intelliot intelliot linked a pull request Sep 7, 2023 that will close this issue
APIv2(gateway_balances, channel_authorize): update errors #4618
Merged
intelliot pushed a commit that referenced this issue Sep 21, 2023
@PeterChen13579
APIv2(gateway_balances, channel_authorize): update errors (#4618)
2487dab 
gateway_balances
* When `account` does not exist in the ledger, return `actNotFound`
  * (Previously, a normal response was returned)
  * Fix #4290
* When required field(s) are missing, return `invalidParams`
  * (Previously, `invalidHotWallet` was incorrectly returned)
  * Fix #4548

channel_authorize
* When the specified `key_type` is invalid, return `badKeyType`
  * (Previously, `invalidParams` was returned)
  * Fix #4289

Since these are breaking changes, they apply only to API version 2.

Supersedes #4577
ckeshava pushed a commit to ckeshava/rippled that referenced this issue Sep 22, 2023
@PeterChen13579 @ckeshava
APIv2(gateway_balances, channel_authorize): update errors (XRPLF#4618)
05ac9c9 
gateway_balances
* When `account` does not exist in the ledger, return `actNotFound`
  * (Previously, a normal response was returned)
  * Fix XRPLF#4290
* When required field(s) are missing, return `invalidParams`
  * (Previously, `invalidHotWallet` was incorrectly returned)
  * Fix XRPLF#4548

channel_authorize
* When the specified `key_type` is invalid, return `badKeyType`
  * (Previously, `invalidParams` was returned)
  * Fix XRPLF#4289

Since these are breaking changes, they apply only to API version 2.

Supersedes XRPLF#4577
sophiax851 pushed a commit to sophiax851/rippled that referenced this issue Jun 12, 2024
@PeterChen13579 @sophiax851
APIv2(gateway_balances, channel_authorize): update errors (XRPLF#4618)
d094c42 
gateway_balances
* When `account` does not exist in the ledger, return `actNotFound`
  * (Previously, a normal response was returned)
  * Fix XRPLF#4290
* When required field(s) are missing, return `invalidParams`
  * (Previously, `invalidHotWallet` was incorrectly returned)
  * Fix XRPLF#4548

channel_authorize
* When the specified `key_type` is invalid, return `badKeyType`
  * (Previously, `invalidParams` was returned)
  * Fix XRPLF#4289

Since these are breaking changes, they apply only to API version 2.

Supersedes XRPLF#4577
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
API Change Will Need Documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

APIv2(gateway_balances, channel_authorize): update errors PeterChen13579/rippled
2 participants
@intelliot @mounikakun