View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005538 | unreal | ircd | public | 2020-01-22 07:31 | 2020-01-26 09:56 |
| Reporter | k4be | Assigned To | syzop | ||
| Priority | normal | Severity | tweak | Reproducibility | always |
| Status | closed | Resolution | no change required | ||
| Product Version | 5.0.1 | ||||
| Summary | 0005538: Inconsistent format of WHOIS 311 reply | ||||
| Description | The format changes when the reply is coming through from different server - last argument is missing the starting ":" character. Some clients may have problems handling this (as i had to tweak mine for it). whois nick :localserver.example.com 311 test nick ident host * :realname whois nick nick :remoteserver.example.com 311 test nick ident host * realname | ||||
| Tags | No tags attached. | ||||
| 3rd party modules | |||||
|
|
I have not fixed this bug because I don't believe it is a bug. I have however fixed something very related. So thanks for the report. See the changelog below: commit c9feaa0b1538026669d9f2105300df38dfea4ee3 (HEAD -> unreal50, origin/unreal50) Author: Bram Matthys <[email protected]> Date: Sun Jan 26 09:46:59 2020 +0100 Fix : at beginning of gecos being dropped off in intra-WHOIS. This does NOT "fix" https://bugs.unrealircd.org/view.php?id=5538: WHOIS nick :localserver.example.com 311 test nick ident host * :realname WHOIS nick nick :remoteserver.example.com 311 test nick ident host * realname .. because your IRC protocol parser should not care about a : or a lack of :. For text not containing spaces nor :-prefix there is no difference in meaning and it should parse to the same. However, this DOES fix an issue if the realname itself started with a colon, such as "USER x x x ::something": WHOIS nick :localserver.example.com 311 test nick ident host * ::something WHOIS nick nick :remoteserver.example.com 311 test nick ident host * :something .. because that does not have the same meaning and is a real incorrect drop of a character. Yeah, I took into account spaces, but not a word starting with :, my bad. |
