[Bug 21467] New: StreamTube: make StreamTubeNewConnection useful, and also rename it

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue Apr 28 18:47:37 CEST 2009


           Summary: StreamTube: make StreamTubeNewConnection useful, and
                    also rename it
           Product: Telepathy
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: telepathy-spec
        AssignedTo: telepathy-bugs at lists.freedesktop.org
        ReportedBy: simon.mcvittie at collabora.co.uk
            Blocks: 20835

+++ This bug was initially created as a clone of Bug #20835 +++

OfferStreamTube should not take an access_control_param since none of them make
any sense at all in this context.

StreamTubeNewConnection should take a variant, containing the
access-control-mechanism-specific access_control_param, like:

* for Port: the port the CM will connect from

* for Credentials: an arbitrary byte identifying which of several possibly
concurrent connections this is (changing the definition of Credentials so it's
not necessarily '\0')

* for Localhost, Netmask: there is no suitable thing, so the service cannot
distinguish between different remote users (but they can't reliably distinguish
between different users of the same machine either)

... except that someone needs to work out whether there is a point at which the
byte can be re-used.

Also, the name is rubbish. NewStreamTubeConnection? NewConnection? New?

(Indeed, stream and D-Bus tubes are distinct types now that we're in the
glorious future, so Offer, Accept would be better method names.)

Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the telepathy-bugs mailing list