Kalle Olavi Niemitalo wrote:
> Michael Sinz <Michael.Sinz@sinz.org> writes:
>
>
>>Anyway, the problem is that the ":" character is not escaped in the
>>"href" attribute of the file and directory elements.
>>
>>This is a problem as the ":" character is a special character that
>>separates protocol from port within a URL and thus must be escaped
>>if it is not serving in that manner.
>
>
> Are the "href" attributes in the XML output even documented to be URLs?
BTW - It is not just in the XML output that this is a problem. The same
problem exists in the HTML output. The following (see below) HTML was
generated by mod_dav_svn when no XSLT is defined and clicking on the link
for the "TestCase-:-/" link - since it now looks like a "TestCase-" protocol
(vs http:) and a path of "-/"
So, the problem is real and exists even in the basic mod_dav_svn HTML output
in addition to the XML output.
PS - Note that the XML href is escaped specifically for URL/URI use vs the
name is escaped for XML/HTML use. The same is true in the HTML output as
can be seen below.
-----------------------------------------------------------------------------------
<html><head><title>Revision 31: /trunk/tests</title></head>
<body>
<h2>Revision 31: /trunk/tests</h2>
<ul>
<li>..</li>
<li>.svn_index</li>
<li>TestCase- -/</li>
<li>TestCase-!-/</li>
<li>TestCase-"-/</li>
<li>TestCase-#-/</li>
<li>TestCase-$-/</li>
<li>TestCase-%-/</li>
<li>TestCase-&-/</li>
<li>TestCase-'-/</li>
<li>TestCase-(-/</li>
<li>TestCase-)-/</li>
<li>TestCase-*-/</li>
<li>TestCase-+-/</li>
<li>TestCase-,-/</li>
<li>TestCase---/</li>
<li>TestCase-.-/</li>
<li>TestCase-:-/</li>
<li>TestCase-;-/</li>
<li>TestCase-<-/</li>
<li>TestCase-=-/</li>
<li>TestCase->-/</li>
<li>TestCase-?-/</li>
<li>TestCase-@-/</li>
<li>TestCase-[-/</li>
<li>TestCase-\-/</li>
<li>TestCase-]-/</li>
<li>TestCase-^-/</li>
<li>TestCase-_-/</li>
<li>TestCase-`-/</li>
<li>TestCase-mv/</li>
<li>TestCase-{-/</li>
<li>TestCase-|-/</li>
<li>TestCase-}-/</li>
<li>TestCase-~-/</li>
<li>TestCase--/</li>
<li>mkdirs.pl</li>
</ul>
<hr noshade><em>Powered by Subversion version 1.2.3 (r15833).</em>
</body></html>
--
Michael Sinz Technology and Engineering Director/Consultant
"Starting Startups" mailto:michael.sinz@sinz.org
My place on the web http://www.sinz.org/Michael.Sinz
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 22 05:31:42 2005