Improve SSH key article #161

Merged
fnetX merged 3 commits from update-ssh-article into master 4 months ago
n commented 4 months ago
Collaborator

This PR improves the SSH key article, removes the part where email is used as a label (newcomers might think this is mandatory and leak their email) and changes the public key type to a more secure ed25519.

This PR improves the SSH key article, removes the part where email is used as a label (newcomers might think this is mandatory and leak their email) and changes the public key type to a more secure ed25519.
n added 1 commit 4 months ago
fnetX approved these changes 4 months ago
1. Copy the SSH key to your clipboard. Attention: Copy only the public part of the key not the private one. You can identify it by the `.pub` extension. By default, you can find the public key in `$HOME/.ssh/id_ed25519.pub`.
On Linux you can use the `xclip` command like this
On Linux you can use `xclip` on the command line. You may need to install it from your package manager.
fnetX commented 4 months ago
Poster
Collaborator

Why drop the installation hint? Because it's obvious that you need to install software if it isn't found?

Why drop the installation hint? Because it's obvious that you need to install software if it isn't found?
n commented 4 months ago
Poster
Collaborator

I didn't drop the installation hint..

I didn't drop the installation hint..
fnetX commented 4 months ago
Poster
Collaborator

Ahh, sorry, must have been very tired. Was preparing for a bigger event on Saturday, dunno why I even bothered to review this when I didn't even manage to properly parse the diff :-)

Ahh, sorry, must have been very tired. Was preparing for a bigger event on Saturday, dunno why I even bothered to review this when I didn't even manage to properly parse the diff :-)
fnetX marked this conversation as resolved
n added 2 commits 4 months ago
fnetX merged commit 6839d4119c into master 4 months ago
fnetX referenced this issue from a commit 4 months ago
n deleted branch update-ssh-article 4 months ago

Reviewers

fnetX approved these changes 4 months ago
The pull request has been merged as 6839d4119c.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.