Fix migrate_db_chrysalis_to_stardust() for some ledger nano wallets

iotaledger · Oct 9, 2023 · fc8d1be · fc8d1be
1 parent b3740c2
commit fc8d1be
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/bindings/nodejs/CHANGELOG.md b/bindings/nodejs/CHANGELOG.md
@@ -28,6 +28,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 
 - Added `SeedSecretManager` to `SecretManagerType`;
+- `migrateDbChrysalisToStardust()` for some ledger nano wallets;
 
 ### Removed
 

diff --git a/sdk/CHANGELOG.md b/sdk/CHANGELOG.md
@@ -32,6 +32,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 
 - Update protocol params and addresses with correct bech32 HRP in `Wallet::set_client_options()`;
+- `migrate_db_chrysalis_to_stardust()` for some ledger nano wallets;
 
 ## 1.1.0 - 2023-09-29
 

diff --git a/sdk/src/wallet/migration/chrysalis.rs b/sdk/src/wallet/migration/chrysalis.rs
@@ -319,7 +319,9 @@ pub(crate) mod rocksdb {
                 let value_utf8 =
                     String::from_utf8(value.to_vec()).map_err(|_| Error::Migration("invalid utf8".into()))?;
                 // "iota-wallet-key-checksum_value" is never an encrypted value
-                if key_utf8 == "iota-wallet-key-checksum_value" {
+                // "FIRST_LEDGER_ADDRESS" was at some point not re-encrypted with the correct password and should
+                // therefore also be ignored to not return an error
+                if key_utf8 == "iota-wallet-key-checksum_value" || key_utf8 == "FIRST_LEDGER_ADDRESS" {
                     value_utf8
                 } else if let Ok(value) = serde_json::from_str::<Vec<u8>>(&value_utf8) {
                     decrypt_record(value, encryption_key)?