Wilfredo Sánchez Vega | 1 Dec 2006 07:07
Favicon

Re: DAV:read privilege and browsing

   Right.  This is how, for example, a typical UNIX file system works.

	-wsv

On Nov 30, 2006, at 8:57 AM, Julian Reschke wrote:

> Basically, if the server exposes the names of children that the user  
> doesn't have access to, security works in a different way. For  
> instance, users will have to move resources they don't want to be  
> visible into a specific folder, and deny read access to that folder  
> as well.

Attachment (smime.p7s): application/pkcs7-signature, 2746 bytes
Wilfredo Sánchez Vega | 1 Dec 2006 07:18
Favicon

Re: DAV:read privilege and browsing


On Nov 30, 2006, at 12:32 AM, Julian Reschke wrote:

>>   I obviously shouldn't be able to read (all of?) the child's  
>> properties, but there is some merit to wanting to be able to see  
>> that the child's URI is present, even if I can't read the child's  
>> properties
>
> Right.
>
>> or content.  I might even want to expose the DAV:resource-type  
>> property so you can tell if it's a collection, etc.
>
> I don't think RFC3744 would allow the latter, even though I would  
> consider it harmless...

   My read leads to the same conclusion.

>>   This also nominally affects GET, when I'm rendering a directory  
>> listing of the parent.  I'd like to show all children, but if you  
>> aren't allowed to see them in PROPFIND, it makes sense that they  
>> should be hidden from the rendered listing as well.
>
> Correct. I personally think they should appear in both, potentially  
> marked up as non-accessible (greyed out...).

   OK, that's where I was heading.  Cyrus had the same concern as  
Kevin; that the file name may itself contain sensitive information,  
and basically cited the same "FIRE-KEVIN.doc" example.  :-)  I'm  
willing to live with that, though; there is plenty of precedent there  
(Continue reading)

Lisa Dusseault | 1 Dec 2006 20:16
Favicon

Re: draft 16 vs issue bz238


Yeah, the section reference is wrong, I can fix that.  I'll continue  
to use my own wording for the rest as it's not quite true that  
there's two different formats (I can think of three: empty body,  
'error' body or 'multistatus').

Lisa

On Nov 28, 2006, at 5:36 AM, Julian Reschke wrote:

>
> From <http://ietf.osafoundation.org:8080/bugzilla/show_bug.cgi? 
> id=238#c2>:
>
> Draft 16 added:
>
> "If a server attempts to make any of the property changes in a  
> PROPPATCH request (i.e. the request is not rejected for high-level  
> errors before processing the body), the response MUST be a Multi- 
> Status response (Section 13)."
>
> That's incorrect. The whole point is that in this case, the Multi- 
> Status reponse format is *not* the one defined in Section 13. So  
> please replace with the previously suggested text:
>
> "The response to a PROPPATCH request can be in two different  
> formats: should the server reject the request altogether (because  
> of missing access rights, failed conditional headers, malformed  
> request syntax, etc.), the status SHOULD be non-2xx HTTP status. On  
> the other hand, if the server did attempt the property  
(Continue reading)

Julian Reschke | 1 Dec 2006 20:22
Picon
Picon

Re: draft 16 vs issue bz238


Lisa Dusseault schrieb:
> 
> Yeah, the section reference is wrong, I can fix that.  I'll continue to 
> use my own wording for the rest as it's not quite true that there's two 
> different formats (I can think of three: empty body, 'error' body or 
> 'multistatus').

So then why does Section 13 say 
(<http://greenbytes.de/tech/webdav/draft-ietf-webdav-rfc2518bis-16.html#rfc.section.13>):

---
A Multi-Status response uses one out of two distinct formats for 
representing the status:

    1. A 'status' element as child of the 'response' element indicates 
the status of the message excecution for the identified resource as a 
whole (for instance, see Section 9.6.2). Some method definitions provide 
information about specific status codes clients should be prepared to 
see in a response. However, clients MUST be able to handle other status 
codes, using the generic rules defined in Section 10 of [RFC2616].
    2. For PROPFIND and PROPPATCH, the format has been extended using 
the 'propstat' element instead of 'status', providing information about 
individual properties of a resource. This format is specific to PROPFIND 
and PROPPATCH, and is described in detail in Section 9.1 and Section 9.2.
---

??

Best regards, Julian
(Continue reading)

Lisa Dusseault | 1 Dec 2006 20:31
Favicon

Re: draft 16 vs issue bz238


These two statements are not inconsistent.  There are two formats for  
Multi-status responses; there are at least three legal formats for  
PROPFIND responses.  A PROPFIND response is not always a Multi-Status  
response (in the case of errors).

Lisa

On Dec 1, 2006, at 11:22 AM, Julian Reschke wrote:

> Lisa Dusseault schrieb:
>> Yeah, the section reference is wrong, I can fix that.  I'll  
>> continue to use my own wording for the rest as it's not quite true  
>> that there's two different formats (I can think of three: empty  
>> body, 'error' body or 'multistatus').
>
> So then why does Section 13 say (<http://greenbytes.de/tech/webdav/ 
> draft-ietf-webdav-rfc2518bis-16.html#rfc.section.13>):
>
> ---
> A Multi-Status response uses one out of two distinct formats for  
> representing the status:
>
>    1. A 'status' element as child of the 'response' element  
> indicates the status of the message excecution for the identified  
> resource as a whole (for instance, see Section 9.6.2). Some method  
> definitions provide information about specific status codes clients  
> should be prepared to see in a response. However, clients MUST be  
> able to handle other status codes, using the generic rules defined  
> in Section 10 of [RFC2616].
(Continue reading)

Cullen Jennings | 1 Dec 2006 21:01
Picon
Favicon
Gravatar

Re: Draft -16 out now


On Nov 29, 2006, at 5:00 AM, Julian Reschke wrote:

>
>
> Issue 244: <http://ietf.osafoundation.org:8080/bugzilla/ 
> show_bug.cgi?id=244> (lock creator vs owner)
>

Thought I might like to see this fixed, when I looked at this with my  
chair hat on, it did not seem like failing to fix this was going to  
have a significant chance of causing non interoperable implementations

> Issue 251:
> <http://ietf.osafoundation.org:8080/bugzilla/show_bug.cgi?id=251>  
> (lock root being a URI, not a resource)

On this one, it seems that the issue occurs in many places in the  
text and is one of the main areas of unresolved consensus around  
locking. If you think we should fix it in this one place using your  
proposed text, I see no objections to that and glad to do it. If you  
want it fixed everywhere, I think that will require reopening the  
unresolved issues on locking and I view that as a 6 month to 2 year  
project.

I also agree the use of section 13 instead of 9 in bug 238 is a bug.

I'm trying to get my head around the "dav: error response" thread.

>
(Continue reading)

Lisa Dusseault | 1 Dec 2006 21:20
Favicon

Draft -17 with three changes


I've submitted a new draft based on comments on -16.  XML and text  
versions at http://ietf.osafoundation.org/xythoswfs/webui/webdav/ 
rfc2518bis until the version submitted to the Internet-Drafts  
repository goes up.

Changes section for this version:

G.12.  Changes in -17

    Change reference for PROPFIND MultiStatus response format from
    section 15 to section 9.2.1

    Add another "except" clause to statement requiring pre/postcondition
    codes to appear in 'error'

    Remove requirement to use TLS -- back to requiring channel to be
    secure.

Lisa

Julian Reschke | 1 Dec 2006 21:34
Picon
Picon

Re: Draft -16 out now


Cullen Jennings schrieb:
>> Issue 244: 
>> <http://ietf.osafoundation.org:8080/bugzilla/show_bug.cgi?id=244> 
>> (lock creator vs owner)
>>
> 
> Thought I might like to see this fixed, when I looked at this with my 
> chair hat on, it did not seem like failing to fix this was going to have 
> a significant chance of causing non interoperable implementations

Yes, it would. That's why it's one of the significant changes from RFC2518.

Existing clients rely on the fact that DAV:owner is whatever they passed 
in in the LOCK request, not what "creator" (which is the authenticated 
principal who did the LOCK).

That's something completely different. If we even don't get simple 
things like this right, we really should stop.

>> Issue 251:
>> <http://ietf.osafoundation.org:8080/bugzilla/show_bug.cgi?id=251> 
>> (lock root being a URI, not a resource)
> 
> On this one, it seems that the issue occurs in many places in the text 
> and is one of the main areas of unresolved consensus around locking. If 
> you think we should fix it in this one place using your proposed text, I 
> see no objections to that and glad to do it. If you want it fixed 
> everywhere, I think that will require reopening the unresolved issues on 
> locking and I view that as a 6 month to 2 year project.
(Continue reading)

Geoffrey M Clemm | 1 Dec 2006 22:17
Picon
Favicon

Re: Draft -16 out now


I agree with Julian's points below.  In particular:
- Changing the interpretation of DAV:owner would introduce a significant
interoperability problem.
-There has always been consensus that the root of a lock is a URI, not a
resource, but that text never seems to make it into a draft.

I did review draft 16, and in general, my conclusion is
that without significant change (such as adopting much of the content of
Julian's draft), this draft would do more harm than good, and that it
would be better for the WebDAV community to just issue a brief errata list,
until the problems in the current draft can be fixed.

Cheers,
Geoff

Julian wrote on 12/01/2006 03:34:14 PM:
> Cullen Jennings schrieb:
> >> Issue 244:
> >> <http://ietf.osafoundation.org:8080/bugzilla/show_bug.cgi?id=244>
> > Thought I might like to see this fixed, when I looked at this with my
> > chair hat on, it did not seem like failing to fix this was going to have
> > a significant chance of causing non interoperable implementations

> Yes, it would. That's why it's one of the significant changes from RFC2518.
> Existing clients rely on the fact that DAV:owner is whatever they passed
> in in the LOCK request, not what "creator" (which is the authenticated
> principal who did the LOCK).


> That's something completely different. If we even don't get simple
> things like this right, we really should stop.
>
> >> Issue 251:
> >> <http://ietf.osafoundation.org:8080/bugzilla/show_bug.cgi?id=251>
> >> (lock root being a URI, not a resource)
> >
> > On this one, it seems that the issue occurs in many places in the text
> > and is one of the main areas of unresolved consensus around locking. If
> > you think we should fix it in this one place using your proposed text, I
> > see no objections to that and glad to do it. If you want it fixed
> > everywhere, I think that will require reopening the unresolved issues on
> > locking and I view that as a 6 month to 2 year project.
>
> Cullen, sorry, but again this is not something where there was any
> disagreement. This is not an unresolved issue at all. The root of the
> lock is a URI, not a resource. If it would be the resource, locking and
> (multiple) bindings wouldn't interoperate.
>
> If you really think we didn't reach consensus on these points, then I
> really think we can give up hope on an RFC2518 revision which at least
> clarifies locking. Right now, it resolves some problems, and adds some more.
>
> > I also agree the use of section 13 instead of 9 in bug 238 is a bug.
> >
> > I'm trying to get my head around the "dav: error response" thread.
>
> That's admittedly a new issue, but probably can be resolved relatively
> simply.
>
> > ...
> >>> Once publication is requested, Ted as the AD would have the
> >>> opportunity to review and ask us to correct problems and it would go
> >>> for IETF LC. This would be another opportunity to identify any true
> >>> show stopper problems that absolutely needed to be fixed.
> >>> Cullen <with me chair hat on>
> >>
> >> My experience is that comments sent during IETF Last Call aren't
> >> tracked, and the people submitting them frequently get zero feedback
> >> from the authors and the IESG (last experienced with XCAP). That being
> >> said, I don't think it's a good quality check for a document anymore.
> >>
> >
> > Hmm - I know some times it seems this way but the IESG usually takes a
> > pretty close look at Last Call comments. My vague recollection of the
> > XCAP work was their was significant discussion of your comments and that
> > the document was updated to address some of them. I also seem to recall
> > that Lisa with her AD role actually took some of your comments and added
> > them as DISCUSS against the document. I'm not claiming for a second that
> > the handling of XCAP was flawless or that the end result does not have
> > significant problems, but I do think your comments were heard.
>
> The problem with the IESG LC process in general is that it completely
> lacks visibility. You don't know whether people read your comments, any
> you usually do not get any feedback at all, until it is too late (that
> is, the IESG approved the document). Fix that process issue, and IESG
> Last Call actually will become a useful test for a spec.
>
> Best regards, Julian
>
Manfred Baedke | 2 Dec 2006 00:14
Picon
Favicon

Re: Draft -16 out now

I definitely agree with Julian and Geoff.
Obviously, Julian has put quite some effort in merging the results of the WG last call reviews into his draft. I have difficulties to understand why this progress should be thrown away.

Regards,
Manfred

Geoffrey M Clemm wrote:

I agree with Julian's points below.  In particular:
- Changing the interpretation of DAV:owner would introduce a significant
interoperability problem.
-There has always been consensus that the root of a lock is a URI, not a
resource, but that text never seems to make it into a draft.

I did review draft 16, and in general, my conclusion is
that without significant change (such as adopting much of the content of
Julian's draft), this draft would do more harm than good, and that it
would be better for the WebDAV community to just issue a brief errata list,
until the problems in the current draft can be fixed.

Cheers,
Geoff

Julian wrote on 12/01/2006 03:34:14 PM:
> Cullen Jennings schrieb:
> >> Issue 244:
> >> <http://ietf.osafoundation.org:8080/bugzilla/show_bug.cgi?id=244>
> > Thought I might like to see this fixed, when I looked at this with my
> > chair hat on, it did not seem like failing to fix this was going to have
> > a significant chance of causing non interoperable implementations

> Yes, it would. That's why it's one of the significant changes from RFC2518.
> Existing clients rely on the fact that DAV:owner is whatever they passed
> in in the LOCK request, not what "creator" (which is the authenticated
> principal who did the LOCK).


> That's something completely different. If we even don't get simple
> things like this right, we really should stop.
>
> >> Issue 251:
> >> <http://ietf.osafoundation.org:8080/bugzilla/show_bug.cgi?id=251>
> >> (lock root being a URI, not a resource)
> >
> > On this one, it seems that the issue occurs in many places in the text
> > and is one of the main areas of unresolved consensus around locking. If
> > you think we should fix it in this one place using your proposed text, I
> > see no objections to that and glad to do it. If you want it fixed
> > everywhere, I think that will require reopening the unresolved issues on
> > locking and I view that as a 6 month to 2 year project.
>
> Cullen, sorry, but again this is not something where there was any
> disagreement. This is not an unresolved issue at all. The root of the
> lock is a URI, not a resource. If it would be the resource, locking and
> (multiple) bindings wouldn't interoperate.
>
> If you really think we didn't reach consensus on these points, then I
> really think we can give up hope on an RFC2518 revision which at least
> clarifies locking. Right now, it resolves some problems, and adds some more.
>
> > I also agree the use of section 13 instead of 9 in bug 238 is a bug.
> >
> > I'm trying to get my head around the "dav: error response" thread.
>
> That's admittedly a new issue, but probably can be resolved relatively
> simply.
>
> > ...
> >>> Once publication is requested, Ted as the AD would have the
> >>> opportunity to review and ask us to correct problems and it would go
> >>> for IETF LC. This would be another opportunity to identify any true
> >>> show stopper problems that absolutely needed to be fixed.
> >>> Cullen <with me chair hat on>
> >>
> >> My experience is that comments sent during IETF Last Call aren't
> >> tracked, and the people submitting them frequently get zero feedback
> >> from the authors and the IESG (last experienced with XCAP). That being
> >> said, I don't think it's a good quality check for a document anymore.
> >>
> >
> > Hmm - I know some times it seems this way but the IESG usually takes a
> > pretty close look at Last Call comments. My vague recollection of the
> > XCAP work was their was significant discussion of your comments and that
> > the document was updated to address some of them. I also seem to recall
> > that Lisa with her AD role actually took some of your comments and added
> > them as DISCUSS against the document. I'm not claiming for a second that
> > the handling of XCAP was flawless or that the end result does not have
> > significant problems, but I do think your comments were heard.
>
> The problem with the IESG LC process in general is that it completely
> lacks visibility. You don't know whether people read your comments, any
> you usually do not get any feedback at all, until it is too late (that
> is, the IESG approved the document). Fix that process issue, and IESG
> Last Call actually will become a useful test for a spec.
>
> Best regards, Julian
>


Gmane