Skip to content

Conversation

@ABuljko
Copy link
Collaborator

@ABuljko ABuljko commented Oct 17, 2025

Documentation on SSH Authentication with FIDO2 for GitLab and GitHub.

@ABuljko ABuljko force-pushed the ssh-authentication branch 2 times, most recently from ba4dee0 to 06bd7a2 Compare October 23, 2025 06:35
@ABuljko ABuljko closed this Oct 23, 2025
@ABuljko ABuljko force-pushed the ssh-authentication branch from 06bd7a2 to 8958431 Compare October 23, 2025 06:40
@ABuljko ABuljko reopened this Oct 23, 2025
ABuljko and others added 4 commits October 24, 2025 10:57
@jans23 jans23 requested a review from daringer October 25, 2025 11:42
@jans23
Copy link
Member

jans23 commented Oct 25, 2025

I think we should give a more specific recommendation regarding non-resident or resident key. @daringer please have a look.

@ABuljko ABuljko requested a review from jans23 October 30, 2025 06:39
ABuljko and others added 2 commits October 30, 2025 07:40
@jans23
Copy link
Member

jans23 commented Oct 30, 2025

@daringer please have a look at the CI error.

@daringer
Copy link
Contributor

  • the CI error is expected for an external repository - need to fix that in the workflow to only try running this job for PRs from within the Nitrokey org - so nothing to worry here.
  • @ABuljko please make sure PR consists of only one commit, means squash them all together and ensure it's properly rebased on top of main

otherwise this lgtm

@jans23
Copy link
Member

jans23 commented Oct 30, 2025

Squashing commits is the default here so no need to extra squash it. Shall we ignore the CI error and merge anyways?


.. note::

Resident keys can later be listed and imported on another system with:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see above comment about reasoning for resident keys


2. Open a terminal and create your SSH key. Replace ``"your_comment"`` with a label to identify it (e.g., "Nitrokey GitLab").
By default, the key is a non-resident key, meaning a local key handle is stored in ``~/.ssh/`` while the private key remains securely on the Nitrokey.
Use the ``-O resident`` option if you want the key to be portable across systems.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for now I would not mention any reason for or against resident keys - I'd prefer to have a separate page/article in FIDO2 about Resident-Keys/Passkeys. Please just leave a recommendation that people should use resident-keys for ssh.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, let's leave the "recommendations" out and just describe the different options (like -o resident) and the refer to the page/article which describes the resident-key-advantages/disadvantages

@daringer daringer merged commit 0d2b690 into Nitrokey:main Nov 18, 2025
2 of 3 checks passed
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.

3 participants