Hi Nil,
On Wed, Jul 31, 2013 at 10:52 AM, Nil Croustillac <[email protected]>wrote: > Hi all, > > I faced a special case where I needed to support the OPTIONS method from > the W3C cross-domain specification. > So I've made this patch but I am not sure if this is exactly how it needs > to be handled. > > Could you have look and let me know if you are interested in integrate it. > If this is the case I could send the corresponding patch on Duda IO. > > Best regards, > Nil. > > _______________________________________________ > Monkey mailing list > [email protected] > http://lists.monkey-project.com/listinfo/monkey > > 1) adding OPTIONS makes sense, but the patch do not look to work correctly as its breaking the response headers: HTTP/1.1 200 OK Server: Monkey/1.3.0 Date: Wed, 31 Jul 2013 17:18:27 GMT Connection: Close Allow: GET,POST,HEAD,PUT,DELETE,OPTIONSContent-Length: 0 after Allow is adding a CRLF, and after the last method supported is not adding a new CRLF 2) please submit the patch in GIT format, once you do your changes you can do : $ git commit -a -s -m "HTTP: add support for OPTIONS method" $ git format-patch -1 the last command will generate the GIT patch, submit that file to the mailing list as an attachment. regards, -- Eduardo Silva http://edsiper.linuxchile.cl http://monkey-project.com
_______________________________________________ Monkey mailing list [email protected] http://lists.monkey-project.com/listinfo/monkey
