Re: [PATCH] Add 'Error validating server' section in FAQ
From: Stefan Sperling <stsp_at_elego.de>
Date: Thu, 31 May 2012 21:39:34 +0200
On Fri, Jun 01, 2012 at 12:20:23AM +0530, Jeyanthan wrote:
Thanks for the patch! Review below.
> Index: faq.html
Please don't use upper case letters in the 'href' link.
> +validating server certificate</tt> exception though I configure the SSL certificates correctly in the server.</a></li>
I'd suggest the following wording for this FAQ entry's title:
I get an "Error validating server certificate" error even though SSL
I wouldn't use the term "exception". We call them "errors" in other
> @@ -4188,6 +4190,39 @@ See <a href="http://svn.haxx.se/dev/archive-2011-0
"This error occurs if the certificate issuer ..."
> +by the SVN client
Full stop here?
And make the following the start of a new sentence?
> and will ask you whether you trust the certificate
Maybe say: "Subversion will ask you ...."
> and if you want to add this certificate.<br />
Maybe say "store" or "save", instead of "add"? Otherwise it's not clear
> +<pre style="margin-left: 40px;">$ svn info https://mysite.com/svn/repo<br />Error validating server certificate for 'https://mysite.com:443':<br />- The certificate is not issued by a trusted authority. Use the<br />fingerprint to validate the certificate manually!<br />Certificate information:<br />- Hostname: mysite.com<br />- Valid: from Wed, 18 Jan 2012 00:00:00 GMT until Fri, 18 Jan 2013<br />23:59:59 GMT<br />- Issuer: Google Inc, US<br />- Fingerprint:<br />34:4b:90:e7:e3:36:81:0d:53:1f:10:c0:4c:98:66:90:4a:9e:05:c9<br />(R)eject, accept (t)emporarily or accept (p)ermanently?</pre>
> +<meta http-equiv="content-type" content="text/html; charset=utf-8" />
Why add a <meta> tag here?
This </p> seems to have no opening <p>.
> +<p>In some cases, even if you accept this by enterinp 'p' option, the
Again, I'd prefer "error" instead of "exception".
> +be multiple reasons.
> The problem may be, that the subversion
The above comma is not necessary.
> +configuration directory, which is normally under your ~/.subversion has
Again, superfluous comma. Maybe say "your ~/.subversion directory"?
> +wrong permissions, so that each time you want to permanently add the
Hmmm... I suppose we should consider making 'svn' issue a warning
I'm not sure if this is good advice.
So in your case, the files inside the directory had wrong permissions?
If you had a directory with wrong permissions, it might likewise be a
You added just one <div> tag but close two. That doesn't seem right.
This is an archived mail posted to the Subversion Dev mailing list.