-
Notifications
You must be signed in to change notification settings - Fork 51
nkapp2 doc update #493
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
base: main
Are you sure you want to change the base?
nkapp2 doc update #493
Conversation
The successful documentation build was deployed for preview: |
Nice! pretty good, I'll add some comments inline |
Nitrokey App 2 is the graphical application for Nitrokey 3 devices. For Nitrokey Pro and Nitrokey Storage, use `Nitrokey App 1 <https://www.nitrokey.com/download>`__ instead. | ||
Nitrokey App 2 is the graphical application for Nitrokey 3 devices. For Nitrokey Pro and Nitrokey Storage, use instead. | ||
|
||
The Nitrokey App 2 is a graphical application designed to manage Nitrokey 3 devices. It provides a user-friendly interface for configuring FIDO2 authentication, password storage, firmware updates, and PIN management. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is "password storage" the correct term? In Nitrokey App 1 we called it Password Safe. What is the correct term here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure.
Is password safe is included in the password storage feature ?
Otherwise password storage is a old term and should be changed by password safe
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shoud the header of the passwords file be Password Safe then ?
|
||
To use a password algorithm, go to security settings of the application you want to secure and enable two-factor authentication. Choose the “Authenticator App” or “Manual Setup” option if available. | ||
|
||
The service will show a secret key. Copy this secret key to enter into your credential setup. Open the password manager app and add a new credential. Choose the algorithm you want to use and paste the secret key. Save the credential. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"show a secret key" this is often combined with a QR code, which is a helpful hint here for the instructions.
"Open the password manager app" ?
@@ -23,3 +23,63 @@ Entries are divided into two categories: unprotected (shown as an open lock) and | |||
You can choose to create a protected entry by checking the "Protect with PIN" box when adding a new entry. | |||
|
|||
You also have the option to require user presence in the form of a required touch on the Nitrokey 3 to access the entry by checking the "Require User Presence" box. | |||
|
|||
Store and Manage Various Credentials |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use a more snappy heading.
|
||
.. figure:: ./images/TOTP-code.png | ||
|
||
HOTP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both sectoins TOTP and HOTP are almost identical. Also HMAC is very similar. Maybe it would be better to have a single chapter for all three but describe the differences (which are not here yet) such as when to use which method.
Verification Requirements | ||
************************* | ||
|
||
If a PIN has been set, it must be entered before making any changes. You are allowed only 8 attempts before lockout. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lockout? I think it is irrevocable.
The successful documentation build was deployed for preview: |
no sure about the pin managment section I merged updates with old doc cause some information are still relevant