ip4-address buddy property - still needed?

[Giannis Galanis]

The feature, although not usable by the activities, it has other benefits.

By observing the buddy list, you acquire instant information of the network
connection go the users:
when connected to channel 1 for example:
169.254.x.x address are in link-local
172.18.x.x are connected to schoolserver

when connected to a jabber server:
169.254.x.x are connected through an MPP
18x.x.x are media lab
172.18.x.x are connected to schoolserver in olpc
etc

It is information continuously used in network testing, also useful from the
users prespective:
1. in the case of connecting to multiple jabber servers, the user should be
able to tell which XO in the neighbout view belongs to the same school
2. get the geopraphical location of another user

In future versions of the neighbor view, or through other activities, the
user should be able to filter for specific XOs according to location, or
school(in the case he's connected to many servers). Two children in the same
school should be able to recognize each other even if they are connected
through a jabber server, other then the one in the school.

It can also be useful for locating an XO in case of theft.

I have also added a ticket(4405) for adding the public id in the buddy list
properties.

It is a small part of data(both IPs, private and public), which can be
harmfully incorporated in the telepathy services.

Please let me know if you agree,

yani



On 10/25/07, Jim Gettys <jg at laptop.org> wrote:
>
> It seems, from your discussion like unless someone grumbles today, this
> should be removed immediately.  And it removed within a week, even if
> someone grumbles...
>                              - Jim
>
>
> On Thu, 2007-10-25 at 10:15 +0100, Simon McVittie wrote:
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > We still have one set of OLPC-specific patches to Salut (the link-local
> > collaboration backend) that has been rejected upstream, which is the one
> > that adds support for the deprecated ip4-address buddy property. This
> was
> > used during a transitional period to enable simple TCP-based
> collaboration
> > for activities that didn't use Tubes; Sjoerd is reluctant to keep this
> > patch set, because it's meant to have gone away by now!
> >
> > Is anyone still using this property? If not, can we kill it? It was
> > added in Trial-2, and it was meant to be gone by Trial-3 but was left in
> > just in case, so it really ought to disappear. When it does, we can
> > delete some code from Salut and Presence Service.
> >
> > Places it's exposed in the APIs, which I propose to get rid of:
> >
> > PS D-Bus API: Buddy.GetProperties() returns a dict that contains
> >     "ip4-address": "10.0.0.1" (or whatever), and Buddy.PropertyChanged
> >     signal includes a dict that can contain the same
> >
> > sugar.presence: Buddy has a GLib property "ip4-address" (aka
> >     buddy.props.ip4_address) and can emit it in its property-changed
> signal
> >
> > The Read activity appears to be the only thing in my jhbuild that uses
> > ip4-address (#4297). It should be ported to either stream tubes (when
> they're
> > ready in Salut, which should be this or next week) or D-Bus tubes (now).
> >
> > Gabble already supports stream tubes, so stream-tube support can be
> > implemented on a branch and tested against Gabble. Porting from plain
> TCP
> > to stream tubes should be very straightforward; I hope to produce a
> > proof-of-concept patch for Read later today.
> >
> >     Simon
> > -----BEGIN PGP SIGNATURE-----
> > Version: GnuPG v1.4.6 (GNU/Linux)
> > Comment: OpenPGP key: http://www.pseudorandom.co.uk/2003/contact/ or
> pgp.net
> >
> > iD8DBQFHIF7HWSc8zVUw7HYRAvp6AJ9G/Xiw27pPPMm0g02vhXzRhzUxqwCfW27Z
> > nh1B/wqe7GD/xf/YaOPVaw8=
> > =42L7
> > -----END PGP SIGNATURE-----
> > _______________________________________________
> > Devel mailing list
> > Devel at lists.laptop.org
> > http://lists.laptop.org/listinfo/devel
> --
> Jim Gettys
> One Laptop Per Child
>
>
> _______________________________________________
> Devel mailing list
> Devel at lists.laptop.org
> http://lists.laptop.org/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.laptop.org/pipermail/devel/attachments/20071026/8bc21e64/attachment.html>