cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Francesco Chicchiriccò (Commented) (JIRA) <j...@apache.org>
Subject [jira] [Commented] (COCOON3-95) Sitemap file not validated against schema
Date Wed, 04 Apr 2012 06:45:19 GMT

    [ https://issues.apache.org/jira/browse/COCOON3-95?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13246069#comment-13246069
] 

Francesco Chicchiriccò commented on COCOON3-95:
-----------------------------------------------

Couldn't an example of new act:src usage be added in cocoon-sample's sitemap.xmap?
                
> Sitemap file not validated against schema
> -----------------------------------------
>
>                 Key: COCOON3-95
>                 URL: https://issues.apache.org/jira/browse/COCOON3-95
>             Project: Cocoon 3
>          Issue Type: Bug
>          Components: cocoon-general
>    Affects Versions: 3.0.0-beta-1
>            Reporter: Javier Puerto
>            Assignee: Francesco Chicchiriccò
>             Fix For: 3.0.0-beta-1
>
>         Attachments: SitemapBuilder-COCOON-95.diff, cocoon3-95.txt, sitemap-schema.diff,
sitemap-validation.tar.gz
>
>
> http://cocoon.markmail.org/thread/cq6nrzy5xladcuys
> Summary: Lars Huttar found that his sitemap declaration was not working as expected.
Some matchers worked an others not. Finally the problem was a matcher tag not inside a pipeline
tag.
> Attached is a block to reproduce the problem, I just review the SitemapBuilder class
and there's not validation at all against a schema. So if the sitemap.xmap file is a XML well
formed, C3 will not throw any error about. The ugly issue is that C3 is returning a HTTP status
code of 200 instead of a code 500 and also the exception in the log is not very helpful, NullPointerException.
> I think that we should validate the sitemap file or at least response with the right
HTTP status code and better error information in this case. We can do something like we have
already for the SchemaProcessorTransformer, using the caching to avoid unnecessary processing.
The schema file path is trunk/cocoon-sitemap/src/main/resources/cocoon-sitemap-1.0.xsd.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

Mime
View raw message