Add transparent refund address for shielded Zcash Maya swaps#459
Open
j0ntz wants to merge 1 commit into
Open
Conversation
Contributor
Author
d550dc3 to
67bbce0
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 67bbce0. Configure here.
Maya cannot refund a shielded Zcash source and rejects the refund with "can't refund ZEC from shielded source without a refund address". Add the wallet's transparent (t-address) refund address to the Maya swap memo as DESTADDR/REFUNDADDR so Maya can process refunds. The refund is injected into the memo client-side rather than passed to the quote endpoint. Appending it to the quote request overflows Zcash's 80-char transparent-memo limit and the quote endpoint rejects the swap. The ZEC swap carries its memo in the shielded note, which is not bound by that limit, so the refund is added after the quote is fetched. Non-Zcash sources are unchanged and continue to default to the sending address.
67bbce0 to
dfd5a7a
Compare
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.





CHANGELOG
Does this branch warrant an entry to the CHANGELOG?
Dependencies
none
Description
Asana: https://app.asana.com/0/1215088146871429/1214541361005860
Problem. When Edge sends a shielded-Zcash swap to Maya, the swap memo carries no refund address. Maya cannot refund a shielded source and rejects the refund:
For every chain except Zcash, Maya defaults the refund to the sending address, which is correct. A shielded ZEC source has no transparent sending address to refund to, so the integrator must supply a transparent (t-address) refund address in the swap memo.
Fix. In the shared Thorchain/Maya common code, when the source wallet is Zcash, fetch the wallet's transparent address (
getAddress(fromWallet, 'transparentAddress'), the same helper already used for the ZEC receive side) and pass it asrefund_addressto thequote/swapendpoint. The node bakes it into the swap memo's destination field per the Maya memo spec (SWAP:ASSET:DESTADDR/REFUNDADDR:...). For ZEC the memo then flows into the existing ZIP-321 shielded-memo path unchanged. Non-Zcash sources are untouched and keep defaulting to the sending address (refund_addressis only added when present, so their quote requests are byte-for-byte identical).The change lives entirely in the dependency; no gui-side wiring is required.
Verification.
tsc --noEmit, eslint, andyarn testall pass.mayanode.mayachain.info): the samequote/swaprequest returns memo=:e:0x04c5...withoutrefund_addressand=:e:0x04c5.../bc1q...with it, confirming the node appends the refund address to the destination field exactly as this change relies on.trading is halted, can't process swap) at the protocol level, so no ZEC quote/swap can be produced regardless of wallet funding. Re-run the in-app smoke once Maya resumes ZEC trading.Note
Medium Risk
Changes Maya/Thorchain ZEC swap memo construction and refund routing; mistakes could misdirect refunds, but the change is narrowly scoped to ZEC sources and post-quote memo mutation only.
Overview
Shielded Zcash → Maya swaps now carry a transparent t-address refund in the swap memo so Maya can refund when the source is shielded (it cannot default to the sender like other chains).
For ZEC source wallets, the plugin loads
transparentAddressvia the samegetAddresshelper used elsewhere. The refund is not sent onquote/swap—that would blow past Zcash’s 80-character transparent-memo limit and fail quoting. After the quote returns, the ZEC / ZIP-321 path rewrites memo field 3 fromDESTADDRtoDESTADDR/REFUNDADDRbefore the memo is encoded into the shielded note (which is not subject to that limit). Non-ZEC swaps are unchanged.CHANGELOG documents the Maya behavior under Unreleased.
Reviewed by Cursor Bugbot for commit dfd5a7a. Bugbot is set up for automated code reviews on this repo. Configure here.