matrix-appservice-irc icon indicating copy to clipboard operation
matrix-appservice-irc copied to clipboard

Bridge Matrix threads

Open tadzik opened this issue 3 years ago • 7 comments
trafficstars

This treats each thread reply as an old message reply, as in: always quotes the previous message from a thread, regardless of how long ago it happened – with the possibility of multiple threads going on concurrently, short replies just turn into a mess.

image

It is quite noisy, but I think it's the best we can do to preserve the context. Replies from IRC won't work, obviously :(

tadzik avatar Jan 10 '22 14:01 tadzik

I suppose the jury is out on whether it's useful to have this

This is what it goes down to, yeah. If it'll end up the giving the false impression that we can transparently bridge threads, and yet we can't bridge replies from IRC to Matrix it'll just become a mess. Perhaps it'd make more sense to do channels-as-threads on the IRC side?

tadzik avatar Jan 10 '22 15:01 tadzik

It is quite noisy, but I think it's the best we can do to preserve the context. Replies from IRC won't work, obviously :(

Wouldn't it be best to just support IRCv3 reply client tag as per https://github.com/matrix-org/matrix-appservice-irc/issues/1414?

Mikaela avatar Jan 10 '22 21:01 Mikaela

It is quite noisy, but I think it's the best we can do to preserve the context. Replies from IRC won't work, obviously :(

Wouldn't it be best to just support IRCv3 reply client tag as per #1414?

I realise it's a bit chicken and egg, but atm only IRCCloud supports it...so we're unlikely to do the work to support it until Server/Client impls catch up.

Half-Shot avatar Jan 12 '22 11:01 Half-Shot

It is quite noisy, but I think it's the best we can do to preserve the context. Replies from IRC won't work, obviously :(

Wouldn't it be best to just support IRCv3 reply client tag as per #1414?

I realise it's a bit chicken and egg, but atm only IRCCloud supports it...so we're unlikely to do the work to support it until Server/Client impls catch up.

It's also supported by Limnoria and Palaver has expressed interest towards implementing it. On servers at least Ergo has it and I would think InspIRCd to at least have it planned if not implemented already.

Mikaela avatar Jan 12 '22 11:01 Mikaela

until Server/Client impls catch up.

Servers don't need to implement it. It's a client-tag, and servers just pass all client tags without explicitly implementing them. (Though UnrealIRCd has a whitelist, but it already has the tag on its whitelist.)

progval avatar Jan 12 '22 12:01 progval

until Server/Client impls catch up.

Servers don't need to implement it. It's a client-tag, and servers just pass all client tags without explicitly implementing them. (Though UnrealIRCd has a whitelist, but it already has the tag on its whitelist.)

Mm, though client-tags is still lagging behind a bit as well (https://github.com/solanum-ircd/solanum/issues/167). Granted, we could probably implement given enough servers do support it.

Half-Shot avatar Jan 12 '22 12:01 Half-Shot

I am under impresson that Solanum is going to get a rewrite at some point and is not going to implement a lot of IRCv3 features before it happens, so I wouldn't include them in considerations how widely supported something is.

Mikaela avatar Jan 12 '22 13:01 Mikaela