Merge lp://staging/~adam-delvecchio/bzr/662448-sshkey-doc into lp://staging/bzr

Proposed by Adam Del Vecchio
Status: Work in progress
Proposed branch: lp://staging/~adam-delvecchio/bzr/662448-sshkey-doc
Merge into: lp://staging/bzr
Diff against target: 113 lines (+109/-0)
1 file modified
doc/en/user-guide/ssh_keys.txt (+109/-0)
To merge this branch: bzr merge lp://staging/~adam-delvecchio/bzr/662448-sshkey-doc
Reviewer Review Type Date Requested Status
Martin Pool Needs Fixing
Review via email: mp+38682@code.staging.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
Martin Pool (mbp) wrote :

Thanks!

The ReST markup is a bit wrong: in the wiki the headings have bars next to them whereas in rest they need an underline of the same length as the heading. Example terminal output or files should have a :: on the preceding line; inline examples should be in ``double backticks``. See http://docutils.sourceforge.net/docs/user/rst/quickref.html

Perhaps there are a few more points to make in the introduction:

 * you can authenticate over ssh by a key pair or by a password (or potentially by other mechanisms); the server chooses what it will accept
 * an ssh key is an assymetric key: there is a 'public' part you put on the server and a private part you keep

I think that using a master connection, while really useful to document, is not normally what people want and describing it first may cause counfusion?

+
+In some cases, ssh may refuse to work if your public key is in your ~/.ssh directory.
+
+ mv ~/.ssh/id_rsa.pub ~/id_rsa.pub
+should fix the problem.

I guess that's on the wiki page but I can't imagine why it would work and it seems like a kind of random suggestion.

s/exxe/exe

review: Needs Fixing
Revision history for this message
Martin Pool (mbp) wrote :

oh, also this probably needs to be mentioned in 'index.txt' to be included within the manual

Revision history for this message
Adam Del Vecchio (adam-delvecchio) wrote :

I will fix and re-commit :)

Also, as to the random suggestion, it works (in my experience), merely because on some servers (I've seen it at least twice), ssh doesn't like the public key in the same directory as the private key. Its mentioned a few times in various mailing lists and documents around the web, I believe :)

Revision history for this message
Martin Pool (mbp) wrote :

Hi, I just wanted to check this wasn't blocked on anything. If you'd like me to finish it off, just let me know.

Revision history for this message
Vincent Ladeuil (vila) wrote :

@Adam: Do you need help here (just say so :) ?

Also, I think we need you to execute the Canonical's Contributor Agreement: <http://www.canonical.com/contributors>.

Unmerged revisions

5505. By Adam Del Vecchio

Fixed bug #662448, added an ssh_keys help document

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.