empathy does not allow /list command

Bug #509802 reported by xteejx
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Empathy
Confirmed
Wishlist
telepathy-idle
Confirmed
Medium
telepathy-idle (Ubuntu)
Triaged
Wishlist
Unassigned

Bug Description

Binary package hint: telepathy-idle

When in a room, the /list command does nothing, as it is not supported.
This is a standard IRC command and should be implemented.

Revision history for this message
In , Danielle Madeley (danielle-madeley) wrote :

There is a branch for this in idle. It seemed to work when I tried it.

Revision history for this message
In , Will Thompson (wjt) wrote :

(In reply to comment #1)
> There is a branch for this in idle. It seemed to work when I tried it.

Listing rooms on big IRC servers is a bad idea: you don't get any messages other than the room list until listing finishes. On OFTC and Freenode, when I type '/list' into irssi, it tells me:

  Irssi: Doing this is not a good idea. Add -YES option to command if you really mean it

Not clear what we should do in Idle.

Revision history for this message
xteejx (xteejx) wrote :

Binary package hint: telepathy-idle

When in a room, the /list command does nothing, as it is not supported.
This is a standard IRC command and should be implemented.

Revision history for this message
xteejx (xteejx) wrote :

This has been committed to Brainstorm: http://brainstorm.ubuntu.com/idea/23381/

Revision history for this message
xteejx (xteejx) wrote :

Marked this Wishlist for obvious reasons.
It has been talked about on the GNOME BTS, but /list doesn't appear to have been implemented looking at the attachments.
Adding upstream watch.

Changed in telepathy-idle (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Brian Curtis (bcurtiswx) wrote :

I see that upstream has closed the upstream report as fixed. Marking as fix committed.. Not sure which version the fix went into so not marking as released yet. For obvious reasons please change as necessary.

Changed in telepathy-idle (Ubuntu):
status: New → Triaged
status: Triaged → Fix Committed
Revision history for this message
Brian Curtis (bcurtiswx) wrote :

Teej. I think I see where you are going with this. Please open a bug upstream asking specifically for the /list command, the upstream bug linked here is one for general / commands in IRC for -idle. Link that here instead and I will re-arrange the statuses as necessary.

Revision history for this message
xteejx (xteejx) wrote :

Will do Brian, thank you.

Revision history for this message
Brian Curtis (bcurtiswx) wrote :

found the upstream link to this. Linked

Changed in telepathy-idle (Ubuntu):
status: Fix Committed → Triaged
Changed in telepathy-idle:
status: Unknown → Confirmed
Revision history for this message
In , Brian Curtis (bcurtiswx) wrote :

Some IRC networks frown upon usage of the room list feature. In this case freenode does since it's a large network. I think this bug should be set to won't fix personally.

Revision history for this message
In , ridgerunner7 (ridgerunner7) wrote :

is it possible to add a "-YES" option to Empathy as well?

Revision history for this message
In , Jonathon Jongsma (jonathon-jongsma) wrote :

just for reference, the branch that implements this for idle is here: http://git.collabora.co.uk/?p=user/jonathon/telepathy-idle;a=shortlog;h=refs/heads/room-list

Changed in telepathy-idle:
importance: Unknown → Medium
Changed in empathy:
importance: Unknown → Wishlist
status: Unknown → Confirmed
Revision history for this message
In , Simon McVittie (smcv) wrote :

I'm not sure what the future of this branch is. I discussed it with Sjoerd this morning but we didn't really come to a conclusion: I'd like to either merge it or abandon it before too long, so we can run Idle through a re-indenter.

Only having one room list seems broken: it's a relic from the pre-Requests spec in which we couldn't have multiple "similar" channels, and we should just delete that wording from the spec. In a protocol like IRC where you can only have one room-listing transaction open at a time, the second and subsequent channels should probably just not list anything until it's their turn?

There are two problems with indiscriminate room listing (room lists that return too many results), which jointly prompted the "/list -YES" feature in irssi:

* the server suffers from unwanted load
* you suffer because there's no way to stop the server sending out room names (possibly throttled to n lines per second where n isn't particularly large), and until it's finished, you can't receive any other message

The normal way to list channels in a dedicated IRC client like irssi is with "/list PATTERN", which lists channels matching PATTERN (which is a shell-style glob, I think?). We don't have API for this yet, only for listing every channel. We could add a method you call instead of ListRooms, ListByPattern (s), which takes an unspecified, protocol-specific pattern (a shell-glob on IRC) and lists some implementation-defined superset of that pattern. This is starting to look more like ContactSearch :-)

Sjoerd mentioned that the UI he'd ideally like to see in Empathy is "start typing and channels appear", like the live-search for contacts. One possibility would be for Idle to make ListByPattern fail until at least n characters were typed, and for Empathy to keep calling it once per keypress until it succeeded.

However, it's easy to construct pathological patterns that match a lot of channels but have enough characters in them to look relatively harmless ("******" on any server that uses shell-globs, but that's easy to avoid by not counting metacharacters, if you know all of them; "#debian-" on irc.debian.org, which is harder to avoid).

One possibility would be to have a flag for "you must specify a pattern" and have Empathy not attempt a live search, or to assume that IRC is the only thing that combines a massive user-base with a shaky protocol and have that flag be "the protocol is 'irc'".

Changed in telepathy-idle:
importance: Medium → Unknown
Revision history for this message
Jon Farz (jfarza1) wrote :

It also doesn't allow /msg if anyone wants to fix that

Changed in telepathy-idle:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.