Skip to content

Add "No password" checkbox - #921

Open
naaraxi wants to merge 3 commits into
GameTec-live:mainfrom
naaraxi:fix/436-t55xx-nopwd-hint
Open

Add "No password" checkbox#921
naaraxi wants to merge 3 commits into
GameTec-live:mainfrom
naaraxi:fix/436-t55xx-nopwd-hint

Conversation

@naaraxi

@naaraxi naaraxi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Fixes #920

@github-actions github-actions Bot added the translation changes When something in the translation changes label Jul 10, 2026
@GameTec-live

Copy link
Copy Markdown
Owner

Are you sure that such a long hint text fits on mobile?

Also: Idk if you can call 00000000 removing the password? youre still setting one its just 00000000?

@naaraxi

naaraxi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Fair, but in that case I'll update the firmware with a proper no_password flag as well, to keep consistent.
Should be ready soon.

@GameTec-live

Copy link
Copy Markdown
Owner

What would be the benefit of adding a firmware flag?

@naaraxi

naaraxi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

It's done for RfidResearchGroup/ChameleonUltra#436 , I guess the benefit is the requested feature being added.
My initial PR was for the "no password" feature using 00000000 as the password, intercepting it and removing the password. But, as you mentioned, setting 00000000 as pwd and setting "no password" are conceptually different.
So, a better solution is using a "--no-pwd" flag as the initial person suggested.

Working on adding a checkbox for "no password" to the GUI.

@GameTec-live

Copy link
Copy Markdown
Owner

Correct me if im wrong, but to the best of my knowledge there is no way for a t55 to not have a password set?

@naaraxi
naaraxi force-pushed the fix/436-t55xx-nopwd-hint branch from 92e5773 to b330356 Compare July 10, 2026 20:35
@naaraxi naaraxi changed the title Add hint that entering 00000000 removes the T55XX password Add "No password" checkbox Jul 10, 2026
@naaraxi

naaraxi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Password enforcement is gated by the PWD enable bit in the config block (block 0)
0 -> no password required, 1 -> require password from block 7
I've tested this via the Python CLI, and using a local build of the GUI 😄 (and proxmark)

@GameTec-live

Copy link
Copy Markdown
Owner

Ok, misread smthng then. Yeah, adding a no_password flag would probably be best then? Or maybe you can parse the "empty string" as no password? As then indeed 0 and no password are different.

@naaraxi

naaraxi commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Added a checkbox labeled "no password", that also disables the 'new key' field, same difference I guess :D If anything, it makes the "no password" more intentional

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

Labels

translation changes When something in the translation changes

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

No Password option for T55xx

3 participants