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

fix(core): add missing semicolon into the passphrase choice for TS3 #3477

Merged
merged 2 commits into from
Jan 3, 2024

Conversation

grdddj
Copy link
Contributor

@grdddj grdddj commented Jan 2, 2024

Fixes #3476:

  • adds semicolon character as a passphrase choice

TODO:

  • record the changed UI tests (if any)
  • consider whether to add a changelog entry

@grdddj grdddj requested a review from prusnak as a code owner January 2, 2024 11:32
@grdddj grdddj requested review from mmilata and Hannsek and removed request for prusnak January 2, 2024 11:32
@prusnak
Copy link
Member

prusnak commented Jan 2, 2024

What is the sort order of the special characters? Is it the same as on Trezor Model T?

@grdddj
Copy link
Contributor Author

grdddj commented Jan 2, 2024

What is the sort order of the special characters? Is it the same as on Trezor Model T?

Yeah, the order should be the same - I have somehow omitted the semicolon for TS3 in the past:

["_<>", ".:@", "/|\\", "!()", "+%&", "-[]", "?{}", ",'`", ";\"~", "$^="],

const SPECIAL_SYMBOLS: &str = "_<>.:@/|\\!()+%&-[]?{},\'`\"~$^=*#";

No idea from where the order came from, seems historical reasons

NOTE: on TT, the *# characters are part of the alphabetical part

@Hannsek
Copy link
Contributor

Hannsek commented Jan 2, 2024

@prusnak do you have some position change in your mind?

@prusnak
Copy link
Member

prusnak commented Jan 2, 2024

@prusnak do you have some position change in your mind?

No. I was just curious.

@grdddj grdddj force-pushed the grdddj/tr_missing_semicolon branch from dfcd068 to 58bafae Compare January 3, 2024 08:39
@grdddj grdddj requested a review from matejcik as a code owner January 3, 2024 08:39
@grdddj grdddj merged commit 988ba0b into main Jan 3, 2024
56 of 57 checks passed
@grdddj grdddj deleted the grdddj/tr_missing_semicolon branch January 3, 2024 13:51
@bosomt
Copy link

bosomt commented Jan 4, 2024

QA OK

Info:

  • Suite version: desktop 24.1.1 (c4cf80383d5f554ff118ba507b743074e73216d4)
  • Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) TrezorSuite/24.1.1 Chrome/118.0.5993.129 Electron/27.0.4 Safari/537.36
  • OS: MacIntel
  • Screen: 1512x982
  • Device: Trezor T2B1 2.6.5 regular (revision 988ba0b)
  • Transport: BridgeTransport 2.0.33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Approved
Development

Successfully merging this pull request may close these issues.

";" is missing in on device passphrase keyboard
4 participants