0037 XLS-37d: Improved Concise Transaction Identifier (CTID) #91
Replies: 13 comments 22 replies
-
Can you give an example of how you see an explorer, a developer and a user using this standard? How would we use this? Just some examples might help. |
Beta Was this translation helpful? Give feedback.
-
What does the |
Beta Was this translation helpful? Give feedback.
-
Updated the name from |
Beta Was this translation helpful? Give feedback.
-
All other artifacts (r-address, hashes, ledger index etc) in XRPL are network agnostic. Should CTIDs be network aware? |
Beta Was this translation helpful? Give feedback.
-
Hey, what about putting the network id at the first position? It would allow for indexing in databases (performance) and even fit the acronym, now being "CanonicalTransaction Identifier" - hehehe |
Beta Was this translation helpful? Give feedback.
-
Why not just add a chain id to the transaction itself, which would cause transactions on different chains to always have different hashes? This is what the various EVM chains do to prevent replaying transactions on different chains and to keep hashes unique. |
Beta Was this translation helpful? Give feedback.
-
Stupid question perhaps, but how to I transfer a NetworkId like "testnet" to a int/hex value? |
Beta Was this translation helpful? Give feedback.
-
Here, X represents, Y represents and Z represents is the ledger sequence number, Y is the transaction ID and Z is the network ID. I'd recommend changing the format to either |
Beta Was this translation helpful? Give feedback.
-
CanonicalTransactionIDentifier :) |
Beta Was this translation helpful? Give feedback.
-
@RichardAH Do you mind pushing this spec along through new CONTRIBUTING process that was recently merged? One intention with the new process is that all specs would move through the process in a more overt fashion before code is merged into rippled Of course, this is a somewhat new process, so feedback welcome as we try it out in this repo (possibly as a PR to the CONTRIBUTING.md if you want). Just today, I initiated this process for the CFT spec -- see it in action here. |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
Hello group: per the new XLS Contributing process, it is my opinion that we have reach a "well-refined standard." As such, I propose that we move this discussion to a file (via #111) and work on final changes using additional PRs, for better change-tracking. Please comment here if you would like to object to moving this spec/discussion forward in the process into a DRAFT spec. The official PR in the rippled repository is XRPLF/rippled#4418. (Note that per the Contributing guidelines, moving a spec into the DRAFT state does not mean any kind of endorsement, nor does it mean that this specification will become adopted. It is solely meant as a mechanism to enable better change tracking using PRs.) |
Beta Was this translation helpful? Give feedback.
-
Closing this discussion in order to direct future comments to these PRs:
|
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
An updated version of this spec can be found here: XLS-37: Concise Transaction Identifier CTID.
The earlier version can be found by looking at the edit history.
Beta Was this translation helpful? Give feedback.
All reactions