link rewriting broken when linking to xml source which contain site: links

• Key: FOR-284
• Type: Bug
• Status: Resolved
• Resolution: Incomplete
• Priority: Major
• Assignee: Unassigned
• Reporter: Nicola Ken Barozzi
• Component/s: Launch 'forrest'
• Affects Version/s: 0.6, 0.7, 0.8
• Fix Version/s: 0.7, 0.8

Description

When linking to *.xml files (e.g. to demonstrate a source file) then if that file contains "site:" or "ext:" links, then they are reported as broken, e.g. docs/site:dtd-docs

Comments

[Nicola Ken Barozzi]

Committed a temporary fix by disabling the 'error:' prepending to links not resolves in site.xml.

Does not need to be solved for 0.6 anymore.

[Nicola Ken Barozzi]

Now that I know what this is about I can better describe it.

[Rick Tessner]

It looks like the problem arises only when there is a link reference to the ".xml" file that contains the "site:" or "ext:" references.

For example, if we have a document called "sample.xml" and there is a link in that document of the form <a href="sample.xml">, the "forrest site" will fail with the broken link message.

This is exactly what we have in the samples/sample.xml. There is in the "Presentations" section: <a href="samples.xml">.

Removing that link will result in a "forrest site" completing cleanly. However, that's not the solution.

I'm guessing that the problem resides in the "linkrewriter" transform somewhere.

[Rick Tessner]

I should add a quick note here about FOR-293 as well. It appears that the fix to the declare-broken-site-links.xsl caused some other issues and that's what FOR-293 is about.

[David Crossley]

Rick's explanation sparked a recollection that something was fixed a few months ago which might be related. We used to get error messages, due to those "site:" links via the linked examples in sitemap-ref.html
http://cvs.apache.org/viewcvs.cgi/forrest/trunk/src/core/context/WEB-INF/cli.xconf?root=Apache-SVN

[Rick Tessner]

Seems to be related to FOR-258 (or may even end up being a duplicate of that issue).

[David Crossley]

Changed the issue title and description to better reflect the real problem.

[David Crossley]

This issue was well-described in Issue FOR-156 which was closed with a workaround. This workaround was partially disabled (see above). So now the problem appeared again, but the links were being excluded via cli.xconf so we didn't see any broken site: links.

I can see a way to restore the original workaround and not have its side-effects.

[David Crossley]

The workaround for FOR-156 was so close to working. Its trouble was only that it needed to deal with docs in sub-directories as well as at top-level. This needed one more exclude pattern in cli.xconf

The workaround was re-implemented at http://svn.apache.org/viewcvs?view=rev&rev=161617

In the core sitemap, after the linkrewriter, the transformer declare-broken-site-links.xsl prepends "error:" to definite broken site: or ext: links. The bogus broken link reporting due to this issue 284 then remain. They will commence with string site:** or docs/site:** etc. These URIs are then excluded by entries in cli.xconf

[David Crossley]

The workaround using cli.xconf has been working nicely.

[David Crossley]

This appears to be fixed.

[David Crossley]

Set the issue Resolution to be "Incomplete". We have a workaround in Cocoon's cli.xconf but the real cause of the issue is not fixed.