[Home] [Downloads] [Search] [Help/forum]


Register forum user name Search FAQ

Gammon Forum

[Folder]  Entire forum
-> [Folder]  MUSHclient
. -> [Folder]  Bug reports
. . -> [Subject]  MUSHclient may disconnect before all output has been received

MUSHclient may disconnect before all output has been received

It is now over 60 days since the last post. This thread is closed.     [Refresh] Refresh page


Pages: 1  2 3  

Posted by Nick Gammon   Australia  (22,973 posts)  [Biography] bio   Forum Administrator
Date Reply #15 on Sun 07 Feb 2010 12:29 AM (UTC)
Message
Glad to hear it works. :)

- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top

Posted by Oldjudge   (1 post)  [Biography] bio
Date Reply #16 on Sun 07 Feb 2010 12:51 PM (UTC)
Message
Hello,

not to be pedantic, but 4.48 seems not to use compression on Achaea (possibly due to the telnet rewrite) and therefore the message appears.

Oldjudge
[Go to top] top

Posted by Worstje   Netherlands  (899 posts)  [Biography] bio
Date Reply #17 on Sun 07 Feb 2010 01:48 PM (UTC)
Message
One mans bugfix is another mans bug...

Sorry, couldn't resist. :)
[Go to top] top

Posted by Nick Gammon   Australia  (22,973 posts)  [Biography] bio   Forum Administrator
Date Reply #18 on Sun 07 Feb 2010 07:10 PM (UTC)
Message
Oops, I accidentally disabled that when I was testing that MCCP v1 worked. Fixed in version 4.49.

- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top

Posted by Twisol   USA  (2,257 posts)  [Biography] bio
Date Reply #19 on Sun 07 Feb 2010 07:28 PM (UTC)
Message
Nick, if I could make a suggestion, perhaps instead "officially" releasing these preview versions, you could tag them on GitHub so interested parties can download from there?

'Soludra' on Achaea

Blog: http://jonathan.com/
GitHub: http://github.com/Twisol
[Go to top] top

Posted by Nick Gammon   Australia  (22,973 posts)  [Biography] bio   Forum Administrator
Date Reply #20 on Sun 07 Feb 2010 07:39 PM (UTC)
Message
I was going to push a version for you to test shortly.

- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top

Posted by Twisol   USA  (2,257 posts)  [Biography] bio
Date Reply #21 on Sun 07 Feb 2010 08:39 PM (UTC)
Message
Excellent! I only brought it up because it seems that these are more preview versions than releases intended for general use, and releasing a new version every time a few features are added didn't seem terribly optimal.

'Soludra' on Achaea

Blog: http://jonathan.com/
GitHub: http://github.com/Twisol
[Go to top] top

Posted by Twisol   USA  (2,257 posts)  [Biography] bio
Date Reply #22 on Mon 08 Feb 2010 06:21 PM (UTC)

Amended on Mon 08 Feb 2010 06:23 PM (UTC) by Twisol

Message
I finally got the source to behave, and when I tested, the original problem still remains. So much for that! =/

EDIT: A new problem I just noticed is that when entering my password, the input was echoed, rather than suppressed as it normally is during the same password entry. Something up with the ECHO subnegotiation?

'Soludra' on Achaea

Blog: http://jonathan.com/
GitHub: http://github.com/Twisol
[Go to top] top

Posted by Nick Gammon   Australia  (22,973 posts)  [Biography] bio   Forum Administrator
Date Reply #23 on Mon 08 Feb 2010 07:52 PM (UTC)
Message
First, you mean echoed in the output window? Second, there is an option to disable that in Commands configuration (Ignore "echo off" messages).

It worked OK for me when I tried it with Smaug. If you still think it is wrong please do a packet debug and check what arrived.

I got:


50 61 73 73 77 6f 72 64 3a 20 ff fb 01
P  a  s  s  w  o  r  d  :     IAC WILL 01


- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top

Posted by Twisol   USA  (2,257 posts)  [Biography] bio
Date Reply #24 on Mon 08 Feb 2010 08:05 PM (UTC)

Amended on Mon 08 Feb 2010 08:20 PM (UTC) by Twisol

Message
No, I mean... I had local echo enabled both times. The password entry is protected by ECHO OFF or whatever. On 4.48, the password is correctly hidden, whereas on my compiled 4.49, the password is still echoed visibly. I -want- it to be hidden, since it's a password entry, but the unreleased version seems to disregard the echo-off message (again, under the same settings as my standard 4.48, I copied the compiled EXE into my MUSHclient folder and ran it there)

Packet debug on 4.48 (correct version)
What is your pas   57 68 61 74 20 69 73 20 79 6f 75 72 20 70 61 73
sword? ÿùÿû.       73 77 6f 72 64 3f 20 ff f9 ff fb 01

Sent  packet: removed bytecount at Monday, February 08, 2010, 1:06:34 PM

removed password for post

Incoming packet: 5 (719 bytes) at Monday, February 08, 2010, 1:06:34 PM

ÿü.Password corr   ff fc 01 50 61 73 73 77 6f 72 64 20 63 6f 72 72
ect. Welcome to    65 63 74 2e 20 57 65 6c 63 6f 6d 65 20 74 6f 20
Achaea...Your la   41 63 68 61 65 61 2e 0d 0a 59 6f 75 72 20 6c 61

'Soludra' on Achaea

Blog: http://jonathan.com/
GitHub: http://github.com/Twisol
[Go to top] top

Posted by Nick Gammon   Australia  (22,973 posts)  [Biography] bio   Forum Administrator
Date Reply #25 on Mon 08 Feb 2010 08:42 PM (UTC)
Message
Well it doesn't do it for me. Look:


Enter your character's name, or type new: nick
Password: 
Reconnecting.
Limbo


Note password not echoed.

- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top

Posted by Nick Gammon   Australia  (22,973 posts)  [Biography] bio   Forum Administrator
Date Reply #26 on Mon 08 Feb 2010 08:43 PM (UTC)
Message
Twisol said:

I finally got the source to behave, and when I tested, the original problem still remains. So much for that! =/


What changes did you have to make, if any?

- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top

Posted by Nick Gammon   Australia  (22,973 posts)  [Biography] bio   Forum Administrator
Date Reply #27 on Mon 08 Feb 2010 09:12 PM (UTC)
Message
If it still does it try putting a breakpoint in on the relevant lines.

- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top

Posted by Twisol   USA  (2,257 posts)  [Biography] bio
Date Reply #28 on Mon 08 Feb 2010 11:27 PM (UTC)
Message
Nick Gammon said:

Well it doesn't do it for me. Look:


Enter your character's name, or type new: nick
Password: 
Reconnecting.
Limbo


Note password not echoed.


The fact remains that it does, inexplicably, echo for me.

Nick Gammon said:

Twisol said:

I finally got the source to behave, and when I tested, the original problem still remains. So much for that! =/


What changes did you have to make, if any?

Just compilation/linker changes, nothing that would affect the behavior of the program directly.

Nick Gammon said:

If it still does it try putting a breakpoint in on the relevant line.

I wouldn't know what the "relevant lines" are, unfortunately. I just have the symptoms and a general idea of what's going on, and I didn't change the source behaviorally.

'Soludra' on Achaea

Blog: http://jonathan.com/
GitHub: http://github.com/Twisol
[Go to top] top

Posted by Nick Gammon   Australia  (22,973 posts)  [Biography] bio   Forum Administrator
Date Reply #29 on Tue 09 Feb 2010 (UTC)

Amended on Tue 09 Feb 2010 12:01 AM (UTC) by Nick Gammon

Message
Well, it is to do with telnet echo option, so the first place I would breakpoint is here:


    case TELOPT_ECHO:    // line 297 in telnet_phases.cpp
          if (!m_bNoEchoOff)  // put breakpoint here
              {
              m_bNoEcho = true;
              TRACE ("Echo turned off\n");
              }
          break; // end of TELOPT_ECHO


That at least confirms you get the IAC WILL 1 (which I saw in your packet debug). And then you should see the line "m_bNoEcho = true;" executed.

Then, once you see the password prompt, but before typing it in, put a breakpoint in doc.cpp:


void CMUSHclientDoc::SendMsg(CString strText,   // line 1101 in doc.cpp
                             const bool bEchoIt,
                             const bool bQueueIt,
                             const bool bLogIt)
  {

  // cannot change what we are sending in OnPluginSent
  if (m_bPluginProcessingSent)
    return;

  bool bEcho = bEchoIt;

  // test to see if world has suppressed echoing
 
  if (m_bNoEcho)   // put breakpoint here 
     bEcho = false;


Now when you hit <enter> to send the password you should see the "bEcho = false;" executed.

A little further down in doc.cpp the bEcho is tested:


  if (bEchoIt)  // line 1272 in doc.cpp
    DisplayMsg (str, str.GetLength (), USER_INPUT | (bLogIt ? LOG_LINE : 0));


These line numbers are valid as at the current commit as I am writing this: 9ac78a6b548da633a78adeb25ca9e3887736d31b

- Nick Gammon

www.gammon.com.au, www.mushclient.com
[Go to top] top

The dates and times for posts above are shown in Universal Co-ordinated Time (UTC).

To show them in your local time you can join the forum, and then set the 'time correction' field in your profile to the number of hours difference between your location and UTC time.


78,042 views.

This is page 2, subject is 3 pages long:  [Previous page]  1  2 3  [Next page]

It is now over 60 days since the last post. This thread is closed.     [Refresh] Refresh page

Go to topic:           Search the forum


[Go to top] top

Quick links: MUSHclient. MUSHclient help. Forum shortcuts. Posting templates. Lua modules. Lua documentation.

Information and images on this site are licensed under the Creative Commons Attribution 3.0 Australia License unless stated otherwise.

[Home]


Written by Nick Gammon - 5K   profile for Nick Gammon on Stack Exchange, a network of free, community-driven Q&A sites   Marriage equality

Comments to: Gammon Software support
[RH click to get RSS URL] Forum RSS feed ( https://gammon.com.au/rss/forum.xml )

[Best viewed with any browser - 2K]    [Hosted at HostDash]