~emersion added bug 1 year, 2 months ago

~emersion added downstream 1 year, 2 months ago

~emersion REPORTED FIXED 1 year, 2 months ago

Fixed in 846c99dedcc6 ("Queue WHOIS commands").

~mooff (edited) 1 year, 2 months ago*

I wondered if there's a timeout for the queuing mechanism - it's not guaranteed that a WHOIS command will generate a response, e.g. when the upstream queries remote servers

~emersion 1 year, 2 months ago

Hm, can this really happen? If so we should clarify it in modern-irc. A timeout would be good to have regardless.

~mooff 1 year, 2 months ago

Fwiw a scenario is described here[1]i

[1] https://ircv3.net/specs/extensions/labeled-response.html#client-implementation-considerations

On Mon, Apr 4 2022 at 20:28:05 -0000, ~emersion outgoing@sr.ht wrote:

Hm, can this really happen? If so we should clarify it in modern-irc. A timeout would be good to have regardless.

-- View on the web: https://todo.sr.ht/~emersion/soju/193#event-169079

~emersion 1 year, 2 months ago

I thought servers had a list of in-progress WHOS per client, and would send a synthetic error when needed. But that's not the case.

~emersion 1 year, 2 months ago

https://github.com/ircdocs/modern-irc/pull/179

~emersion 1 year, 2 months ago

Opened https://todo.sr.ht/~emersion/soju/194