Skip to content

wallet: change the derivation path for modified ZIP32#4307

Merged
mergify[bot] merged 5 commits intomainfrom
tomas/modified-modified-zip32
Feb 5, 2025
Merged

wallet: change the derivation path for modified ZIP32#4307
mergify[bot] merged 5 commits intomainfrom
tomas/modified-modified-zip32

Conversation

@tzemanovic
Copy link
Copy Markdown
Collaborator

Describe your changes

Checklist before merging

  • If this PR has some consensus breaking changes, I added the corresponding breaking:: labels
    • This will require 2 reviewers to approve the changes
  • If this PR requires changes to the docs or specs, a corresponding PR is opened in the namada-docs repo
    • Relevant PR if applies:
  • If this PR affects services such as namada-indexer or namada-masp-indexer, a corresponding PR is opened in that repo
    • Relevant PR if applies:

@tzemanovic tzemanovic requested a review from murisi February 4, 2025 10:29
@tzemanovic tzemanovic marked this pull request as ready for review February 4, 2025 10:29
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 4, 2025

Codecov Report

Attention: Patch coverage is 66.66667% with 6 lines in your changes missing coverage. Please review.

Project coverage is 74.19%. Comparing base (759fcfc) to head (ec5fe65).
Report is 72 commits behind head on main.

Files with missing lines Patch % Lines
crates/wallet/src/lib.rs 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4307      +/-   ##
==========================================
+ Coverage   74.06%   74.19%   +0.13%     
==========================================
  Files         345      345              
  Lines      110193   110623     +430     
==========================================
+ Hits        81611    82074     +463     
+ Misses      28582    28549      -33     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tzemanovic tzemanovic added the backport-libs-0.251 Backport libraries to 0.251 maintenance branch label Feb 5, 2025
Copy link
Copy Markdown
Collaborator

@murisi murisi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks!

@tzemanovic tzemanovic added the merge Ready to merge - mergifyio bot will add the PR to merge queue when all checks pass label Feb 5, 2025
mergify bot added a commit that referenced this pull request Feb 5, 2025
mergify bot added a commit that referenced this pull request Feb 5, 2025
@mergify mergify bot merged commit 7c1309c into main Feb 5, 2025
24 of 25 checks passed
@mergify mergify bot deleted the tomas/modified-modified-zip32 branch February 5, 2025 11:54
mergify bot pushed a commit that referenced this pull request Feb 5, 2025
(cherry picked from commit 2b59f55)
mergify bot pushed a commit that referenced this pull request Feb 5, 2025
(cherry picked from commit ec5fe65)
mergify bot added a commit that referenced this pull request Feb 5, 2025
wallet: change the derivation path for modified ZIP32 (backport #4307)
tzemanovic added a commit that referenced this pull request Feb 5, 2025
* maint-libs-0.47:
  changelog: release Namada libs 0.47.1
  Namada libs 0.47.1
  Revert "Merge pull request #4332 from anoma/murisi/test-vectors-with-chain-ids"
  changelog: add #4283
  PoS: emit token transfer event on claim rewards tx
  Added a changelog entry.
  Print the transaction's chain ID in expert mode.
  Rename test_inverse_conversions to make it clear that it now tests something else.
  Added changelog entry.
  Remove the ability to specify which historical epoch amounts are exchanged to.
  Do not query for the inverse conversion when converting to the latest epoch.
  Added changelog entry.
  Update inverse conversion test.
  Now parameterize query_allowed_conversion with the target epoch.
  fixup! deps: use internal crates defined in workspace
  deps: update non-lib crates Cargo.toml to use workspace defs
  config/release: allow to publish released crates
  deps: fix ignored and rm unused default-features flags
  deps: use internal crates defined in workspace
  specify all internal lib crates in workspace
  deps: rm unused borsh-ext
  fixup! changelog: add #4307
  fixup! wallet: change the derivation path for modified ZIP32
  wallet: update the rustdoc warning about modified ZIP32
  changelog: add #4307
  wallet: change the derivation path for modified ZIP32
  Added a changelog entry.
  Split the production of native and non-native conversions into separate functions.
  Move the construction of reward assets out into a separate function.
  Invert the loop and conditional in the reward distribution algorithm.
  Added changelog entry.
  Added a changelog entry.
  Do not exchange balance amounts when querying non-native token balances.
  Use the normed inflation as the precision to get more accurate native rewards.
  Added changelog entry.
  Utilize leftover dust in conversions.
  changelog: add #4299
  update recommended CometBFT to 0.37.15
  changelog
  deps: switch to published ibc-middleware crates
  Update crates/tests/src/e2e/setup.rs
  update middleware
  ibc-rs 0.57.0
  upstream ibc-rs
  updete ibc-middleware
  WIP: replaced errors
  WIP: upgrade ibc-rs
  changelog
  revert Hermes for CI
  add hermes logs
  Revert
  Revert
  fix typo
  checking CI failure
  Hermes for v1.1.0
  make fmt
  pos: extend test case to also cover what happens when rewards are claimed
  Check for a last reward claim in addition to whether the key is in the state before invoking the shortcut
  fix: improve documentation for --epoch option in rewards command to clarify that it only returns accumulated rewards available to claim
  fix: address review suggestions
  [pos] Add unit test for vp::pos::rewards query with epochs
  enhancement: implement read_subspace_val_with_height for mockdb
  [vscode] Add namada_sdk unit test debug launch definition
  changelog: add
  refactor: Amount::from(0) -> Amount::zero()
  slight wording change: for -> at to make it more clear that rewards are as of beginning of epoch
  cargo fmt
  refactor: use existing storage_value query function for looking up past rewards_counter
  [pos] when querying rewards by epoch, use the historical rewards_counter value
  [pos] Add epoch argument to vp::pos::rewards
  switch to published ledger-namada-rs fork
  Changelog: #4289
  Adds more cases to multisig unit test
  Added a changelog entry.
  Make sure the CI installs libudev-dev before building Namada.
  Upgrade the Ledger app Rust library and also upgrade the Ledger app used in the CI.
  Use prop_filter to ensure that random transactions are generated with valid fees.
  Added changelog entry.
  Update hardware wallet test vectors to display transaction fees in normal mode.
  ci/mergify: update for maint-1.1 and maint-libs-0.47
tzemanovic added a commit that referenced this pull request Mar 14, 2025
Namada libs-v0.48.0 major release.

* tag 'libs-v0.48.0': (153 commits)
  Changelog: Release libs 0.48.0
  Namada libs 0.48.0
  Make sure to reset masp_index counter to zero whenever new block is encountered.
  Correct the masp_index once it has been found.
  Reduce cloning of state by updating only the Merkle tree when searching for correct ordering.
  Update commitment tree anchors in benchmarks.
  Order transactions in the cache according to the MASP index instead of the block index.
  Assume that Transactions from the indexer are already ordered.
  Compute the transaction order even when needs_witness_map_update capability is disabled.
  Added a changelog entry.
  Factor out the Transaction order search into a separate function.
  Optimize transaction order search.
  Make client try all Transaction permutations.
  Instrumented code with commitment anchor existence checks.
  Changelog: Release libs 0.47.3
  Namada libs 0.47.3
  changelog: add #4403
  wallet: fix build with "migrations" feature
  gov: fix build with "testing" feature on
  PoS: fix build with "testing" feature on
  ci/benches: increase timeout and use bigger machine
  test: sa003
  test+bench: sa002
  test: SA001
  fix to write toml files
  changelog: add #4284
  sdk: fix breaking change in getrandom wasm build
  deps: cargo update
  deps: move all non-workspace deps into workspace and update
  deps: update wasmparser to wasmer matching version
  deps: update toml
  deps: update sysinfo
  deps: update sha2
  deps: update rpassword
  deps: update rlimit
  deps: update reqwest
  deps: update num256
  clippy: reinstate disallow unwrap_or_default
  deps: update itertools
  deps: update byte-unit
  dep: update bech32
  dep: update base64
  update non-breaking deps
  deps: fix crate vulnerabilities
  changelog: add #4371
  add VERSIONING.md
  Modifies `PartialEq` for `Signer`. `Authorization`s account for the signer when compared
  Fixes bugs in `PartialEq` implementations for `SigningTxData` and `Authorization`
  Changelog #4230
  Removes duplicated sections from transactions before signing or dumping
  Custom implementaion of `PartialEq` for `Authorization`
  Update `PartialEq` for `SigningTxData`
  CI: update the Ledger app version to 3.0.1
  Added a changelog entry.
  Print the transaction's chain ID in expert mode.
  client: warn about min ledger app version when using shielded keys
  changelog: add #4358
  wallet: check min ledger app version for shielded key derivation
  Added changelog entry.
  Update hardware genesis files to reflect shielded keys derived using new algorithm.
  changelog
  revert hermes version for CI
  upstream hermes
  test/e2e: fix the test with HW wallet automation
  Changelog: Release libs 0.47.2
  Namada libs 0.47.2
  move version detection build script from apps_lib into apps
  keep a separate commit for release changelogs
  apps_lib: output artifacts from build script into OUT_DIR
  tx: build tonic artifacts into OUT_DIR
  changelog: add #4347
  sdk/tx: print tx events if any
  declare internal dev-dependencies via path
  changelog: release Namada libs 0.47.1
  Namada libs 0.47.1
  Revert "Merge pull request #4332 from anoma/murisi/test-vectors-with-chain-ids"
  changelog: add #4283
  PoS: emit token transfer event on claim rewards tx
  Added a changelog entry.
  Print the transaction's chain ID in expert mode.
  Rename test_inverse_conversions to make it clear that it now tests something else.
  Added changelog entry.
  Remove the ability to specify which historical epoch amounts are exchanged to.
  Do not query for the inverse conversion when converting to the latest epoch.
  Added changelog entry.
  Update inverse conversion test.
  Now parameterize query_allowed_conversion with the target epoch.
  fixup! deps: use internal crates defined in workspace
  deps: update non-lib crates Cargo.toml to use workspace defs
  config/release: allow to publish released crates
  deps: fix ignored and rm unused default-features flags
  deps: use internal crates defined in workspace
  specify all internal lib crates in workspace
  deps: rm unused borsh-ext
  fixup! changelog: add #4307
  fixup! wallet: change the derivation path for modified ZIP32
  wallet: update the rustdoc warning about modified ZIP32
  changelog: add #4307
  wallet: change the derivation path for modified ZIP32
  Added a changelog entry.
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-libs-0.251 Backport libraries to 0.251 maintenance branch merge Ready to merge - mergifyio bot will add the PR to merge queue when all checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants