Menu

#4 address over certain length fails

0.5.2
closed-fixed
7
2003-04-27
2003-04-24
No

It appears that a hotmail address over a certain length
fails to log in.

In version 0.4, one of my users was able to log in to her
account (strangerthanfriction@hotmail.com) using
just "strangerthanfriction".

In version 0.5.2, I get this:
-ERR username invalid, specify your full address, e.g.
strangerthanfriction@hotmail.com

However, if I specify the full form of that address, I am
denied login. Shorter addresses which I have tested
work just fine -- it's something about the length of the
address that is causing this problem.

As a temporary solution, I have reverted to 0.4.

Discussion

  • David Smith

    David Smith - 2003-04-25

    Logged In: YES
    user_id=569736

    This problem has occured because we modified the code so
    that you can login with either @msn.com or @hotmail.com
    accounts. Unfortunately it seems that long user names with
    the @hotmail.com don't work if we send @hotmail.com along
    with the username to the hotmail servers, but the @msn.com
    must always be there. I've written a patch to 0.5.2 and
    attached it here which should fix this problem, can you test
    it out please. Just copy the username_fix.patch into your
    unpacked hotway source directory and run the following
    command to patch the sources:

    patch -p1 < username_fix.patch

    Tell me how you go. Thanks, Dave.

     
  • David Smith

    David Smith - 2003-04-25
    • labels: 522710 --> login problems
    • milestone: --> 0.5.2
    • priority: 5 --> 7
    • assigned_to: nobody --> courierdave
    • status: open --> open-fixed
     
  • David Smith

    David Smith - 2003-04-25

    fix for long user name logins

     
  • Daniel Drucker

    Daniel Drucker - 2003-04-26

    Logged In: YES
    user_id=41432

    That fixed the problem. Thanks very much!

     
  • David Smith

    David Smith - 2003-04-27
    • status: open-fixed --> closed-fixed
     
  • David Smith

    David Smith - 2003-04-27

    Logged In: YES
    user_id=569736

    This has been merged into the CVS archives and will be part
    of the next release package. Thanks for the bug report and
    verification of solution.

     

Log in to post a comment.