Shame on me. I only had a first look at it and my impression was that it looks really good. But I wanted to test it at least once before committing.
Just a bit patience, Ingo > Ingo, > > Just thought I'd ask you if you got time to look at my patch on Bind support > in WebdavResource. > > Regards, > Ritu > > -----Original Message----- > From: Ingo Brunberg [mailto:[EMAIL PROTECTED] > Sent: Wednesday, June 30, 2004 6:07 PM > To: [EMAIL PROTECTED] > Subject: Re: BIND method > > > > Ok, sorry, got into it, found the documentation page :D. Still sorry for > > polluting the mailing list. > > > > Anyway, it still remains that it reports 201 instead of 403 when the > > store does not support binding. Do you think this is a bug? > > Yes, a 201 should not be returned if the store does not support > bindings. Maybe a 501 would even be more appropraite than a 403. > > > > > ciao, > > Simone Gianni > > > > Simone Gianni wrote: > > > > > Hi all, > > > I'm using slide client library HEAD, and slide server 2.0 . I have > > > currently coded by hand a snippet to test a bind method, following the > > > specifications at > > > > http://ftp.ics.uci.edu/pub/ietf/webdav/collection/draft-ietf-webdav-bind-02. > txt > > > and reading the bind method source code. It SEEMS to work, in that it > > > returns 201 created, but nothing changes on the server : > > > > > > BIND /slide/files/collection/ HTTP/1.1 > > > Authorization: ... > > > User-Agent: Jakarta Commons-HttpClient/2.0final > > > Host: .... > > > Cookie: $Version=0; JSESSIONID=5F9BB5B646C4C234554F2701B6DC40AA; > > > $Path=/slide > > > Content-Length: 219 > > > > > > > > > <?xml version="1.0" encoding="utf-8" ?> > > > <D:bind xmlns:D="DAV:"> > > > <D:segment>test.xml</D:segment> > > > > > > > <D:href>http://myhost:8080/slide/files/other/collection/realfile.xml</D:href > > > > > > > > </D:bind> > > > > > > HTTP/1.1 201 Created > > > Pragma: No-cache > > > Cache-Control: no-cache > > > Expires: Thu, 01 Jan 1970 01:00:00 CET > > > Content-Length: 0 > > > Date: Wed, 30 Jun 2004 01:10:44 GMT > > > Server: Apache-Coyote/1.1 > > > > > > > > > I'm using the default configuration of the slide-tomcat bundle, so > > > txfile store. I've noticed that in StructureImpl, in the method > > > addBinding, there is a check to see if bindings are enabled by > > > configuration and if the store supports bindings. Currently, if this > > > check is negative, the bind is not executed, but the method still > > > returns 201 - Created, since this is set in BindMethod > > > checkPreconditions and there is no else statement. Could this be the > > > case? Does txfile store support bindings? Should bindings be enabled > > > in the slide.properties file explicitly? (I tried this, but same > > > story, however the default configuration seems to be true) And most of > > > all, is it correct for BindMethod to return "201 Created" when the > > > configuration have binds disabled and/or the store does not supports > > > bindings? Shouldn't it return "403 Forbidden" instead? > > > > > > > > > Please let me know, > > > ciao, > > > > > > Simone Gianni --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
