Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #250 - Add boolean option for passphrase #355

Conversation

isaacvr
Copy link
Contributor

@isaacvr isaacvr commented Oct 31, 2019

Fixes #250

Changes:

  • Add with_passphrase option to fsm_getKeyPairAtIndex. This allows you to enable or not the use of a passphrase for mnemonic and, therefore, to use two generations of different addresses, as long as the device has the passphrase option enabled.
  • In other uses of this function (fsm_getKeyPairAtIndex) outside the signing process, the argument with_passphrase is passed as true, because before this change that was the behavior. Note that in the future the selection of passphrase or not, should be considered in the future in these cases.

Does this change need to mentioned in CHANGELOG.md?
yes

Requires testing
yes

Comments about testing , should you have some

@olemis olemis added the firmware MCU firmware label Nov 20, 2019
@olemis olemis assigned adriantpaez and unassigned stdevAlDen Nov 20, 2019
@olemis olemis added this to the v2 milestone Nov 20, 2019
@olemis
Copy link
Collaborator

olemis commented Nov 20, 2019

does this require changes in go library as well ? @stdevAdrianPaez if so what's the PR ?

olemis added 2 commits March 4, 2020 15:48
…e stdevAdrianPaez_t63_edit_tx_messages_for_passphrase_option branch
@olemis olemis modified the milestones: v2, v3 Jan 10, 2021
@adriantpaez adriantpaez removed their assignment Jun 14, 2023
@isaacvr isaacvr closed this Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
firmware MCU firmware
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Passphrase not supported for signing transactions
5 participants