You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@chemistry.apache.org by "Florian Müller (JIRA)" <ji...@apache.org> on 2014/06/13 10:36:02 UTC

[jira] [Commented] (CMIS-811) Support HEAD and OPTIONS methods for Browser & AtomPub Bindings

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

Florian Müller commented on CMIS-811:
-------------------------------------

I have two questions regarding this patch.

* To implement HTTP OPTIONS correctly, the repository capabilities and the Allowable Action have to be taken into account. If a repository doesn't support a certain feature (= returns the HTTP status code 405), OPTIONS shouldn't indicate that the client could do that. Do you have a use case that requires (generic) HTTP OPTIONS support?
* The patch determines the content length for HTTP HEAD requests and sets the Content-Length header. But the Content-Length header is not set in HTTP GET responses. The content is sent in chunks instead. That is, the headers of HEAD and GET would be different, which is not the intention of HTTP HEAD. Is there a reason for that?


> Support HEAD and OPTIONS methods for Browser & AtomPub Bindings
> ---------------------------------------------------------------
>
>                 Key: CMIS-811
>                 URL: https://issues.apache.org/jira/browse/CMIS-811
>             Project: Chemistry
>          Issue Type: Improvement
>          Components: opencmis-client-bindings
>    Affects Versions: OpenCMIS 0.11.0
>            Reporter: Tom Weinschenk
>            Assignee: Florian Müller
>            Priority: Minor
>              Labels: patch
>
> Browser and AtomPub Bindings for Apache Chemistry OpenCMIS do not currently support HTTP HEAD and OPTIONS methods. While not as common as the four primary HTTP REST methods, these methods are worthy of support within Apache Chemistry OpenCMIS.
> Pull request at https://github.com/apache/chemistry-opencmis/pull/6



--
This message was sent by Atlassian JIRA
(v6.2#6252)