[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Simple] RE: Comments Event Filter Function -01
Hisham you wrote:
> You would propagate it to Bob's presence server. The RLS is
> not the PS (although they can be co-located).
>
Co-location. Thats what I had in mind all along.
Thanks for the clarification.
Rgds/gf
> -----Original Message-----
> From: hisham.khartabil at nokia.com [mailto:hisham.khartabil at nokia.com]
> Sent: Friday, July 16, 2004 4:00 AM
> To: George Foti (QA/EMC); simple at ietf.org
> Subject: RE: Comments Event Filter Function -01
>
>
>
>
> > -----Original Message-----
> > From: ext George Foti (QA/EMC) [mailto:george.foti at ericsson.com]
> > Sent: 15.July.2004 16:24
> > To: Khartabil Hisham (Nokia-TP-MSW/Helsinki); simple at ietf.org
> > Subject: RE: Comments Event Filter Function -01
> >
> >
> > Comments inline Hisham
> > /gf
> > > Removing a filter is done in one way. Replacing a filter is
> > > done in another way. Two filters to the same user is an
> > > error. Having those 2 filters appear in the same request or
> > > in separate requests does not make a difference. We shouldn't
> > > overload the functionality. If something is clearly an error
> > > (trying to replace a filter in a different way than is
> > > specified), then the client must be told about it.
> > >
> > I dont get it.
> > To modify an existing filter for a resource. I have 2
> > possible alternatives here :
> > 1) Use the same filterid with a new content.
> > 2) Use a new filterid that implicitly removes the old filter
> > for that user and replace it with the new filter. That
> > respects the rule of having one filter per resource.
> >
> > You seem to say that 2 is an error, and we should do that in
> > 2 steps, first an explict remove, then add the new filter.
>
> What you are asking for is a filter replacement. For this, it
> is defined that the same filter-id is used with new filter
> contents. It's just 1 step.
>
> Adding new filters is done using new filter-ids. If this new
> filter has a URI that conflicts with another filter URI, then
> it is an error.
>
> So:
>
> 1. Removing is done using the remove attribute
> 2. Replacing/modifying is done by re-using the filter-id
> 3. Adding is done by creating new filter with new filter id
>
> > Can U elaborate on the rationale for that.
> > Does it causes confusion for other cases
> >
> > > >
> > > > Next, in the third paragraph, you have the example on
> > > > myboddies at mydomain.com & bob at mydomain.com, it seems to me
> > > > that the domain must be different for the RLS to extract the
> > > > filter and propagate it, so Bob should have a different
> > > > domain for the statement to apply.
> > >
> > > The example is correct. A filter is not propagated if Bob was
> > > not part of the list, but is part of domain.com.
> >
> > Your statement is true but thats not what the text in the
> > draft is saying.
>
> There is text for that:
>
> "There may be
> situations where the filters were not propagated because
> they address
> a URI that in under the adminstrative domain of the RLS,
> but are not
> part of the resource list that the subscription was
> addressed to. In
> this case, it is the RLS responsibility to make sure than those
> filters are applied to notifications issued."
>
> I'll make this text more normative.
>
> > If Bob was on the list and part of the domain, I dont see
> > where to would you propagate the filter.
>
> You would propagate it to Bob's presence server. The RLS is
> not the PS (although they can be co-located).
>
> >
> >
> >
> > I had one comment I forgot to mention.
> > In section 5.2.1 you talk about mismatch between the R-RI in
> > SUBSCRIBE and URI in the filter.
> > It would help to expand what mismatch means.
> > I know you cover it in several places in the document.
> > Optionally refer to the section where one can find the
> pertinent text.
> >
> >
>
> Added some text about this in all sections that imply URI
> matching is needed.
>
> Thanks,
> Hisham
>
_______________________________________________
Simple mailing list
Simple at ietf.org
https://www1.ietf.org/mailman/listinfo/simple