Github user ushachar commented on the pull request:

    https://github.com/apache/trafficserver/pull/270#issuecomment-128261744
  
    I'm not too sure about this -- It's one thing to allow/block unknown 
methods that parse properly (TS-1232/TS-1407), but adding non-HTTP methods to 
the HTTP method list just for the ip-allow functionlity seems like another 
thing that can be best accomplished in plugin space....
    
    Some technical comments:
    1) Maybe rename HTTP_WKSIDX_... -> WEBDAV_WKSIDX_...
    2) Some WebDAV methods should be treated like POST in that they are 
required to have payload (I don't think there's any method that's explicitly 
forbidden from having payload).
    3) Should we support the "If" header? 
(http://www.webdav.org/specs/rfc4918.html#rfc.section.10.4.5)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to