Skip to content

Fix: Migration 541 reads wrong key in _migrateNonWehAuthentSecureKey#1296

Open
prjanitor wants to merge 1 commit into
archethic-foundation:devfrom
prjanitor:prjanitor/9553fc68b1bef75ce2432d89367631abbec8f817
Open

Fix: Migration 541 reads wrong key in _migrateNonWehAuthentSecureKey#1296
prjanitor wants to merge 1 commit into
archethic-foundation:devfrom
prjanitor:prjanitor/9553fc68b1bef75ce2432d89367631abbec8f817

Conversation

@prjanitor

Copy link
Copy Markdown

Bug Description

The _migrateNonWehAuthentSecureKey() function in migration 541 was incorrectly reading from kVaultSecureKey instead of kNonWebAuthenticationSecureKey. This caused it to:

  1. Read the same key as _migrateVaultSecureKey()
  2. Potentially overwrite the wrong destination
  3. Fail to migrate the intended NonWebAuthent secure key data

Fix

Changed the source key in _migrateNonWehAuthentSecureKey() from kVaultSecureKey to kNonWebAuthenticationSecureKey so it correctly reads the NonWebAuthent secure key for migration.

Files Changed

  • lib/application/migrations/541.dart - Fixed line 31 to read from correct source key

This PR was generated by PRJanitor — an automated tool that finds and fixes small bugs in open-source projects.

We respect your contribution guidelines — if your project doesn't accept bot PRs, we won't send more. You can also add a .github/prjanitor.yml file with enabled: false to opt out explicitly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant