bugWeeChat - Bugs: bug #36825, Weechat sends ::1 incorrectly for...

 
 

bug #36825: Weechat sends ::1 incorrectly for the servername with USER command

Submitter:  Florian Bruhin <the_compiler>
Submitted:  Tue 10 Jul 2012 09:38:38 PM UTC
Votes: 3
 
Category:  irc plugin Severity:  3 - Normal
Item Group:  irc protocol Status:  Fixed
Privacy:  Public Assigned to:  flashcode
Originator Name:  Open/Closed:  Closed
Release:  * 0.3.8 IRC nick:  The-Compiler
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 15 Jul 2012 07:55:53 AM UTC, comment #3: 

Old format was:

USER username username address :real name


New format is:

USER username 0 * :real name


And now spaces are automatically replaced by underscores in username (since no space is allowed here).

Sebastien Helleu <flashcode>
Group administrator
Fri 13 Jul 2012 08:39:40 AM UTC, comment #2: 

Hi,

I just have read again the RFC1459 and RFC2812, both says that this argument is unused (used only for connection between two servers, according to RFC1459).

So I will just send "*" here instead of an address or a hostname (as suggested in RFC2812).

Sebastien Helleu <flashcode>
Group administrator
Tue 10 Jul 2012 10:11:28 PM UTC, comment #1: 

In various other cases (such as some server-server protocols), "0::1" is usually used because it represents the same address as "::1" while avoiding protocol breakage. (This of course applies to all IPv6 addresses starting with ::, which is just a bunch of 0000's collapsed.)

But with the USER command coming for clients, most ircds simply ignore the 2nd and 3rd fields, so anything that does not start with a ":" would be acceptable (for example, "[::1]" or just "*").

grawity <grawity>
Tue 10 Jul 2012 09:38:38 PM UTC, original submission:  

When I connect to ::1 via IPv6, this is what weechat sends:

USER compiler compiler ::1 :Florian Bruhin

This means the value ":1 :Florian Bruhin" is taken as the server name. According to grawity in IRC, 0::1 would be the correct thing to send.

Florian Bruhin <the_compiler>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by flashcode (Posted a comment)
  • -email is unavailable- added by grawity (Voted in favor of this item)
  • -email is unavailable- added by grawity (Posted a comment)
  • -email is unavailable- added by the_compiler (Submitted the item)
  •  

    There are 3 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-09-29 flashcode Open/ClosedOpen Closed
    2012-07-15 flashcode StatusIn Progress Fixed
    2012-07-13 flashcode StatusNone In Progress
        Assigned toNone flashcode
    2012-07-10 grawity Carbon-Copy- Added grawity

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code