Re: s_err.c: Standardize 256 (RPL_ADMINME) with RFC 1459

Thanks Juuso - beyond the RFC issue, have you run into any cases of this causing problems with clients?
Hi Aaron. Let's assume this is the raw conversation, compliant with RFC 1459: ADMIN :irc.se.partyvan.eu 256 * irc.se.partyvan.eu :Administrative info :irc.se.partyvan.eu 257 * :Stockholm, Sweden :irc.se.partyvan.eu 258 * :Partyvan NOC :irc.se.partyvan.eu 259 * :noc@partyvan.eu I've tested this a little and so far found a minor regression with weechat 1.3: -- | irc.se.partyvan.eu :Administrative info -- | Stockholm, Sweden -- | Partyvan NOC -- | noc@partyvan.eu I've heard ZNC also has the same little regression: * irc.se.partyvan.eu :Administrative info * Stockholm, Sweden * Partyvan NOC * noc@partyvan.eu irssi 0.8.17 works perfectly fine! 15:03 -!- irc.se.partyvan.eu Administrative info 15:03 -!- Stockholm, Sweden 15:03 -!- Partyvan NOC 15:03 -!- noc@partyvan.eu I've not tested with HexChat, XChat or other IRC clients yet, but I'm interested to hear about them too. I intend to send patches to IRC client developers for major IRC clients which do not follow the RFC syntax, but I've not yet had time to do so. Of course, it would be interesting to get every IRC client and server closer to the standards and help with future adaptation for a new, up to date IRC standard (IRCv3? Harmony?). For what it's worth, this is the progress on different IRC server daemons: - Charybdis: Merged (e0160ec5a4cbd855751571824a0dbf93aae29790) - nefarious2: Merged (5dce7c3a7c1e08b4d68de78a764af83d6e217375) - InspIRCd: Intent to merge (after fixing a parameter issue) - UnrealIRCd: wontfix (bug tracker #4519) - Bahamut: You are here Other IRCds I've not yet looked into, but will try to do so tomorrow. DBoyz also did some testing before anything was merged. It seems different IRCds use(d) different non-standard 256 replies: bahamut-2.0.7 <- :arcor.de.eu.dal.net 256 DBoyz :Administrative info about arcor.de.eu.dal.net charybdis-3.5.0-rc1 <- :irc-us3.alphachat.net 256 DBoyz :Administrative info about irc-us3.alphachat.net InspIRCd-2.0 <- :lazer.techtronix.net 256 DBoyz :Administrative info for lazer.techtronix.net u2.10.12.14+Nefarious(2.0.0) <- :Baltimore.MD.US.ExileNET.Org 256 DBoyz :Administrative info about Baltimore.MD.US.ExileNET.Org UnrealIRCd-4.0.0 <- :irc.blockah.com 256 DBoyz :Administrative info about irc.blockah.com I also checked the CVS history of Undernet's ircu. The non-standard "Administrative info about <server>" string has been there since the initial commit in year 1999 - so I found no reason why they would have changed it away from the standard.
participants (1)
-
Juuso Lapinlampi