Gammon Forum
Entire forum
MUSHclient
Bug reports
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 page
Pages: 1
2 3
Posted by
| Nick Gammon
Australia (22,973 posts) 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 | top |
|
Posted by
| Oldjudge
(1 post) 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 | top |
|
Posted by
| Worstje
Netherlands (899 posts) 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. :) | top |
|
Posted by
| Nick Gammon
Australia (22,973 posts) 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 | top |
|
Posted by
| Twisol
USA (2,257 posts) 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 | top |
|
Posted by
| Nick Gammon
Australia (22,973 posts) 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 | top |
|
Posted by
| Twisol
USA (2,257 posts) 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 | top |
|
Posted by
| Twisol
USA (2,257 posts) 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 | top |
|
Posted by
| Nick Gammon
Australia (22,973 posts) 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 | top |
|
Posted by
| Twisol
USA (2,257 posts) 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 | top |
|
Posted by
| Nick Gammon
Australia (22,973 posts) 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 | top |
|
Posted by
| Nick Gammon
Australia (22,973 posts) 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 | top |
|
Posted by
| Nick Gammon
Australia (22,973 posts) 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 | top |
|
Posted by
| Twisol
USA (2,257 posts) 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 | top |
|
Posted by
| Nick Gammon
Australia (22,973 posts) 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 | 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:
1
2 3
It is now over 60 days since the last post. This thread is closed.
  Refresh page
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.