Official Shielded Support for Zcash In Ledger HW Wallet

[ainhoa-zondax]

Yes, this is correct! We have been making good progress and have released a new version that is currently being tested by Ledger.

Specifically, we were concerned about ensuring that this new app would enable the unlocking of funds for users who had previously used the non-shielded Zcash app. We have successfully run several tests and were able to unlock funds via our adapted ZEC Wallet Lite and the new Zcash Shielded app. These results are very promising, and if Ledger confirms them through their testing, we will be a big step closer to a public release.

Additionally, we have added support in the new app for the newly released device, Flex.

[dismad]

@ainhoa-zondax

if Ledger confirms

@ainhoa-zondax

will be a big step closer

Tired of the lip service. You guys should have a fire under your ass, and you don’t. Full stop. I’m beginning to think this will never actually be complete. Waste of time, money, and resources. I’m glad they got that FLEX though! …

[Blazin8888]

Once the app is fully released this will not work in Ledger Live correct?

Did Ledger say if they would add a shielding feature in Ledger Live natively or will we have to use Zashi and other 3rd party apps to use this?

[pacu]

@ainhoa-zondax and rest of the Zondax team:

I am pleased to tell you that I’ve picked every bit of the Zondax to ZingoLib PR and updated it to the newest codebase and it can be rebased normally WIP - Grab Zondax's feat/ledger branch and port the changes by pacu · Pull Request #1352 · zingolabs/zingolib · GitHub so that your team can keep working on it.

[ainhoa-zondax]

@pacu

ed every bit of the Zondax to ZingoLib PR and updated it to the newest codebase and it can be rebase

Thanks @pacu for the contribution to ZingoLib. Earlier this year we worked on laying the groundwork for integrating Ledger support with a wallet like Zingo, but the next steps are now in Zingo’s hands. See my previous post from March Official Shielded Support for Zcash In Ledger HW Wallet - #48 by ainhoa-zondax

We’re available to assist with the integration if needed and have previously published a high level integration guide Wallet integration Guide of Zcash-shielded Ledger app - HackMD

[ainhoa-zondax]

Hi @Blazin8888 your understanding is correct: adapting the current Ledger Live integration to be compatible with the new app would require several more months of additional work, which would further delay the release.

While we have experience with Ledger Live integrations, the initial scope of this project was to create a shielded app that could then be integrated with third-party wallets.

[pacu]

@ainhoa-zondax

We’re available to assist with the integration if needed

I have no problem carrying on with this effort provided that ZGC approves my backlog change. Sent you a DM because this PR port needs review from its original authors

[ainhoa-zondax]

I’m really sorry to hear that, but this is also to us the most frustrating project we have had so far.

We assembled a dedicated task force to debug the app, which resulted in significant improvements. We resubmitted the app to Ledger on July 31st for review, incorporating numerous enhancements and adjustments. You can see the PR here, which has already been merged by Ledger New Release by emmanuelm41 · Pull Request #3 · LedgerHQ/app-zcash-new · GitHub

[pacu]

having this merged is a great step forward!

I’ve been using my Nano S with ZecWallet and I notice that the Zcash app actually asks the user to verify a lot more of information that no Zcash wallet actually shows, like the Nullifier. I assume this is a requirement from Ledger. Is there a way to avoid asking the user to “approve” a nullifier? can we skip this? If not, well, I guess that we’ll have to give in and make wallets show the nullifiers on screen to make the users be able to truthfully approve them.

[hanh]

@pacu

I guess that we’ll have to give in and make wallets show the nullifiers on screen to make the users be able to truthfully approve them.

The user is not supposed to trust the wallet because it could have been compromised. That would defeat the purpose of user validation.

[pacu]

I think you are missing my point. I’m wondering why we have to and what is the purpose of showing a piece of information on the hardware wallet that the user can’t actually verify anywhere?

Has anyone seen a nullifier in their lives ? (Core Devs and @hanh answers don’t count )

[hanh]

I don’t think I am missing your point. I am saying that the “workaround” is not acceptable.

Even if I know what a nullified is, there is no way for me to know that it is the right value.

[pacu]

Oh! That’s what I’m saying!

What’s the point of it being there? Is ledger asking for it ? Or is something that can be changed without being kicked out of their walled garden ?

[hanh]

It shouldn’t be a requirement. Bitcoin users are not required to verify their tx inputs. Why should we?