Skip to content

Feat: crypted password property - #3924

Open
achoqt wants to merge 2 commits into
ControlSystemStudio:masterfrom
achoqt:feat-crypted-password-property
Open

achoqt wants to merge 2 commits into
ControlSystemStudio:masterfrom
achoqt:feat-crypted-password-property

Conversation

@achoqt

@achoqt achoqt commented Sep 16, 2026

Copy link
Copy Markdown

Add a new property to provide a hashed password.

Currently, the password is readable in the bob file.

This feature enables one to input a hashed password using three possible hash algorithms (MD5, SHA-256, SHA-512).

In runtime, it will decrypt the password according to the selected hash algorithm.

See screenshot below:

crypted_password_phoebus
  • Testing:
    • generate a hashed password using MD5 or SHA-256, or SHA-512
    • enter the hashed password in one of the widgets which has the password property
    • select the hash algorithm you used to hash the password
    • run the .bob and test to enter the initial password

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
16.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@achoqt

achoqt commented Sep 16, 2026

Copy link
Copy Markdown
Author

Two checks failed.

  • SonarCloud Code Analysis: duplication code.

Indeed, some lines are duplicated. I just added them the same way as in the existing code in the corresponding files.

  • Lint:

I am not sure to understand the errors, and if I do need to fix them myself.

@georgweiss

Copy link
Copy Markdown
Collaborator

@achoqt, could you please provide some context here as to the need/requirements?

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.

2 participants