Skip to content

Commit

Permalink
CI: Don't test zcash_client_backend/lightwalletd-tonic on WASM
Browse files Browse the repository at this point in the history
Broken until hyperium/tonic#1783 is addressed.
  • Loading branch information
str4d committed Jul 12, 2024
1 parent e5517b6 commit 270eec2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ jobs:
run: cargo add --no-default-features --path ../crates/zcash_proofs
- name: Add zcash_client_backend as a dependency of the synthetic crate
working-directory: ./ci-build
run: cargo add --features lightwalletd-tonic --path ../crates/zcash_client_backend
run: cargo add --path ../crates/zcash_client_backend
- name: Copy pinned dependencies into synthetic crate
run: cp crates/Cargo.lock ci-build/
- name: Add target
Expand Down
2 changes: 2 additions & 0 deletions zcash_client_backend/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ and this library adheres to Rust's notion of
### Changed
- MSRV is now 1.70.0.
- Migrated to `tonic 0.12`.
- The `lightwalletd-tonic` feature flag no longer works on `wasm32-wasi` due
to https://github.com/hyperium/tonic/issues/1783.
- `zcash_client_backend::{fixed,standard,zip317}::SingleOutputChangeStrategy`
now implement a different strategy for choosing whether there will be any
change, and its value. This can avoid leaking information about note amounts
Expand Down

0 comments on commit 270eec2

Please sign in to comment.