Skip to content

feat(ripple): XRP memo field for THORChain routing#241

Open
BitHighlander wants to merge 3 commits into
developfrom
feat/ripple-memo
Open

feat(ripple): XRP memo field for THORChain routing#241
BitHighlander wants to merge 3 commits into
developfrom
feat/ripple-memo

Conversation

@BitHighlander

Copy link
Copy Markdown
Owner

Summary

  • Adds optional memo field (proto field 7) to RippleSignTx
  • Serializes XRPL Memos array (STArray[9]/STObject[10]/MemoData VL[13]) after destination in canonical XRPL field order
  • Displays memo on confirmation screen before signing
  • Bumps deps/device-protocol to include the new proto field

THORChain swap memos (e.g. =:ETH.ETH:0x...) are stored as raw UTF-8 bytes in the XRPL MemoData VL field, per the XRPL binary format spec.

Test plan

  • CI green
  • XRP payment with memo signs and broadcasts correctly
  • XRP payment without memo still works (backwards compat)

Add optional memo field (proto field 7) to RippleSignTx.

Firmware changes:
- messages-ripple.options: add RippleSignTx.memo max_size:200
- ripple.c: serialize XRPL Memos array (STArray[9]/STObject[10]/MemoData VL[13])
  after destination in canonical XRPL field order
- fsm_msg_ripple.h: display memo on confirmation screen before signing

THORChain swap memos (e.g. '=:ETH.ETH:0x...') are stored as raw UTF-8
bytes in the XRPL MemoData VL field, following the XRPL binary format spec.
deps/device-protocol → 8ef74da (feat(ripple): memo + THORChain routing)
deps/python-keepkey  → bf870e6 (7.14.2: XRP memo + EVM depositWithExpiry + msg-signing)
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

Successfully merging this pull request may close these issues.

1 participant