View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0005330||unreal||ircd||public||2019-07-13 15:45||2019-08-18 15:21|
|Target Version||Fixed in Version|
|Summary||0005330: finish labeled-response implementation|
|Description||I did start a labeled-response implementation when starting on U5 dev back in April.|
It still needs to be improved to become an acceptable/workable implementation, in particular if there are already tags in the message (easy) and if there already is a BATCH involved (more serious work)
Oh and update it now that IRCv3 has decided what to do with 0 length responses. The current draft spec is here https://github.com/ircv3/ircv3-specifications/blob/master/extensions/labeled-response.md
Since it's an experimental feature, I'm going to work on other things first, that have a higher priority.
|Tags||No tags attached.|
|3rd party modules|
It mostly works right now, but will always issue a BATCH (except for emtpy replies, in which case it will issue an ACK). I'm sure there are corner issues, in fact there are some TODO/FIXME's spread around the source and on my list. But the basic idea is there. My last commit summarizes the current status and plans:
Author: Bram Matthys <email@example.com>
Date: Sun Aug 18 09:24:43 2019 +0200
Enough updates on labeled-response and echo-message for today.
Note that the labeled-response implementation currently requires
'batch' and will always start a BATCH if there is any response.
Later on we can implement a simple queue so we don't have to
start a batch for 1-line responses (which works, but looks a bit
silly if you look at raw server traffic). That may be after alpha1,
though, as there are more (important) things to work on right now.
|2019-07-13 15:45||syzop||New Issue|
|2019-07-13 15:45||syzop||Status||new => assigned|
|2019-07-13 15:45||syzop||Assigned To||=> syzop|
|2019-07-13 15:45||syzop||Relationship added||child of 0005279|
|2019-07-13 15:45||syzop||Relationship added||child of 0005280|
|2019-08-18 15:17||syzop||View Status||private => public|
|2019-08-18 15:20||syzop||Note Added: 0020821|
|2019-08-18 15:21||syzop||Note Edited: 0020821||View Revisions|