Bug 26172 - Incorporate Gabble's decloaking API or something similar
Summary: Incorporate Gabble's decloaking API or something similar
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-spec (show other bugs)
Version: git master
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 70382
  Show dependency treegraph
 
Reported: 2010-01-22 08:21 UTC by Simon McVittie
Modified: 2014-02-04 11:19 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
remove Decloak extension (31.80 KB, patch)
2014-02-04 10:00 UTC, Guillaume Desmottes
Details | Splinter Review

Description Simon McVittie 2010-01-22 08:21:48 UTC
The changes made in Bug #25989 introduced a simple Gabble-specific API for "decloaking" presence. We should have a proper API for this.
Comment 1 Simon McVittie 2013-10-14 15:38:06 UTC
Attachment #87475 [details] pulls Decloak1 into tp-spec.

I wrote:

> Not sure whether to merge this one or not... it's pretty XMPP-specific, but
> then, Conn.I.Cellular is protocol-specific and we still spec'd that one
> centrally.

Guillaume replied:

> I'm not sure either about this one. Is it actually used these days?

The answer to that is: Call1 never did the "automatic decloak attempt" that StreamedMedia did, and we've removed StreamedMedia now, so Decloak won't be used automatically. It'll only be used if someone uses the D-Bus API to ask for it specifically.

Also, xmpp.org changed the namespace from "temppres" back to "decloak", so we're no longer XEP-draft-compliant (although in any case the namespace isn't official until the XEP progresses to draft).

I'd be OK with removing this functionality and putting it back later if there's actually demand, tbh.
Comment 2 Guillaume Desmottes 2014-02-04 10:00:21 UTC
Created attachment 93354 [details] [review]
remove Decloak extension
Comment 3 Simon McVittie 2014-02-04 11:07:50 UTC
Comment on attachment 93354 [details] [review]
remove Decloak extension

Review of attachment 93354 [details] [review]:
-----------------------------------------------------------------

++
Comment 4 Guillaume Desmottes 2014-02-04 11:19:13 UTC
Merged to next.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.