Research solutions to workaround xrpl-py/issues/632

[godexsoft]

See XRPLF/xrpl-py#632
Clio only supports validated ledgers and even provides a separate document describing ledger command functionality.
However, if there is a mix of rippled and clio nodes on s1/s2 then clients can have difficulties parsing the response as it may be different for rippled and clio. Let's try to find a solution to this (even if it's fixing response parsing on the library side).

[cindyyan317]

The issue in XRPLF/xrpl-py#632 seems not exist anymore.

INFO:__main__:Ledger(ledger_hash=None, ledger_index=None, method=<RequestMethod.LEDGER: 'ledger'>, id=None, full=False, accounts=False, transactions=False, expand=False, owner_funds=False, binary=False, queue=False, type=None)
INFO:__main__:Response(status=<ResponseStatus.ERROR: 'error'>, result={'error': 'notSupported', 'error_code': 75, 'error_message': 'Operation not supported.', 'type': 'response', 'request': {'method': 'ledger', 'params': [{'full': False, 'accounts': False, 'transactions': False, 'expand': False, 'owner_funds': False, 'binary': False, 'queue': False}]}}, id=None, type=<ResponseType.RESPONSE: 'response'>)

Clio has fixed this issue in 2.0. Previously, even "full", "accounts","queue" is false, clio will respond "notSupport". Currently , clio will forward the request to rippled if these fields are 'true'.

But clio indeed has different response structure as rippled. Rippled's response contains two sub node "closed", "open". Clio can not obtain open ledger, so there is only one layer in Clio's response. Clio should be friendly to the existing frontend code. Clio can add "closed" sub node.

[godexsoft]

We can add "closed" but that would change our existing API. I'm not sure there is another way though.

[Bronek]

@godexsoft there will be change in API when rippled version 2.0 is released. Since we will need to release clio with matching API changes, this seems like a good opportunity to unify the interface between the two.