Zcash × Brave WalletPersonal project
Search/ Updated
Menu

Type to search. Use ↑ ↓ and Enter to open a result.

Refactor: Remove separate process for Zcash protobuf parsing

Open Issue General Zcash support

Lead item brave-browser#47850 · opened by kdenhartog · last source update

This issue is open on GitHub. Fixes sometimes land through pull requests that do not link the issue (for example a later refactor); this tracker follows only explicit links (closing references, “Resolves” lines, brave_<issue> branch names, uplifts and the issue timeline).

Issue state

Open

Open.

Implementation

No pull request is linked by closing keywords, GitHub’s closing references, a brave_<issue> branch name or the issue timeline.

Release notes

No platform changelog lists this item.

Validation

Not QA-tested (QA/No).

No QA pass labels.

QA labels are Brave’s own validation records per OS. They are not independent verification by this tracker.

Target

No milestone set.

Platforms (OS labels): none

Owner: cypt4

Build presence

Whether a merged pull request in this group is an ancestor of the brave-core tag of each platform’s current build (from versions.brave.com). Code presence is not the same as a feature being exposed on that platform.

PlatformReleaseBetaNightly
DesktopUnknown 1.97.59no merged PR is linked, so presence in this build is unknownUnknown 1.98.47no merged PR is linked, so presence in this build is unknownUnknown 1.100.5no merged PR is linked, so presence in this build is unknown
AndroidUnknown 1.97.57no merged PR is linked, so presence in this build is unknownUnknown 1.98.52no merged PR is linked, so presence in this build is unknownUnknown 1.100.5no merged PR is linked, so presence in this build is unknown
iOSUnknown 1.96.62no merged PR is linked, so presence in this build is unknownUnknown 1.99.44no merged PR is linked, so presence in this build is unknownUnknown 1.100.7no merged PR is linked, so presence in this build is unknown

Members and their history

Issue

brave-browser#47850 Refactor: Remove separate process for Zcash protobuf parsing

State: open · created · updated · discovered via label:feature/web3/wallet/zcash, search:zcash · matched label:feature/web3/wallet/zcash, title:zcash, body:zcash

priority/P4 QA/No dev-concern release-notes/exclude feature/web3/wallet/core feature/web3/wallet/zcash

Description Summary Remove the BraveWalletUtilsService and revert the separate process implementation for Zcash protobuf parsing introduced in PR https://github.com/brave/brave-core/pull/21627 Background PR #21627 moved Zcash protobuf parsing to a separate process citing Chromium's Rule of 2 security guidelines. However, the Rule of 2 documentation includes an exception for protobufs, making this additional complexity unnecessary. Proposed Solution Revert the changes from…

Timeline (3 of 6 events)
  1. 2025-07-22 22:48 Label added priority/P4 by kdenhartog
  2. 2025-07-22 22:48 Label added QA/No by kdenhartog
  3. 2025-07-22 22:48 Label added release-notes/exclude by kdenhartog