mbbsemu / MBBSEmu

The MajorBBS Emulation Project is an Open Source, Cross-Platform emulator for easily running The MajorBBS & Worldgroup Modules
https://www.mbbsemu.com
MIT License
128 stars 14 forks source link

[MBBSEmu] RLogin - Worldgroup 3.x - Sends set of characters on connect #130

Closed BlaZZZed1980 closed 3 years ago

BlaZZZed1980 commented 3 years ago

Module Information

Describe the bug Whenever a Rlogin connection is initiated from a Worldgroup 3.x BBS's Rlogin client, "ss(ANSI up arrow)Pdd" is sent into the emulated module as though the user typed it. This is a known issue with the WG3.x Rlogin client.

To Reproduce Steps to reproduce the behavior:

  1. Connect to a Worldgroup 3.x BBS
  2. Initiate a Rlogin connection to a MBBSEmu emulated module

Expected behavior Would like to see MBBSEmu ignore these characters when a connection is initiated.

Screenshots image

Software Information:

enusbaum commented 3 years ago

Thanks for reporting this!

The fix here might be ensuring the INPUT value is empty when first calling the STTROU during an Rlogin session. This would discard any extra data that came in along with the Rlogin session beyond the actual handshake.

I have a WG 3.3 setup I can test this with and report back 👍

BlaZZZed1980 commented 3 years ago

I would also be happy to install any experimental build that attempts to correct this issue and report back with the results. I have changed my configuration to Rlogin directly to the MBBSemu menu (where the unwanted characters still appear) but at least then when you enter the modules, everything works normally.

synacktic commented 3 years ago

I just added a pull request that I think will fix this. It just depends on if someone knows something I don't about "extra" handshake bits.

synacktic commented 3 years ago

@BlaZZZed1980 could you please try one of these builds: https://www.mbbsemu.com/Downloads/master/021121 and let us know here if that fixes the issue for you?

BlaZZZed1980 commented 3 years ago

Will do....at work now but will try when I get home. The extra stuff I believe is WG sending screensize info in the header instead of how it's supposed to do it. At least that's how it's been explained to me. I will test this build shortly and report back.

On Thu, Feb 11, 2021 at 1:35 PM synacktic notifications@github.com wrote:

@BlaZZZed1980 https://github.com/BlaZZZed1980 could you please try one of these builds: https://www.mbbsemu.com/Downloads/master/021121 and let us know here if that fixes the issue for you?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mbbsemu/MBBSEmu/issues/130#issuecomment-777738021, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARBCF3JENF3LKHVTLCGGXFDS6QWOZANCNFSM4RQLRHJA .

-- Brian J. Morgan President BlaZZZed Lighting blazzze@gmail.com

TQPS commented 3 years ago

I just tested it for him and it is still occuring.

To be clear, this is using RLOGIN with a -a -m switch and mbbsemu-win-x64-021121-4.zip

![image](https://user-images.githubusercontent.com/66294837/107704568-9107a100-6d11-11eb-9cb8-a7b855cff125.png)

The following was logged to Audit trail:

2021-02-12 08:56:16.4618 Info MBBSEmu.Server.Socket.SocketServer.OnNewConnection Accepting incoming Rlogin connection from 192.168.10.203:1023...
2021-02-12 08:56:16.4618 Info MBBSEmu.HostProcess.MbbsHost.AddSession Session 192.168.10.203:1023 added to incoming queue
2021-02-12 08:56:16.4618 Info MBBSEmu.Session.Rlogin.RloginSession.ProcessIncomingByte Rlogin For User: Frank.Dux
2021-02-12 08:56:16.4618 Debug MBBSEmu.Session.Rlogin.RloginSession.ProcessIncomingClientData Ignoring extra rlogin data: "           "
2021-02-12 08:56:16.4618 Info MBBSEmu.HostProcess.MbbsHost.WorkerThread Added Session 192.168.10.203:1023 to channel 0
2021-02-12 09:02:38.6158 Info MBBSEmu.HostProcess.MbbsHost.RemoveSession Removing Channel: 0
synacktic commented 3 years ago

Well boo. I was hoping that was a two birds one stone thing.

BlaZZZed1980 commented 3 years ago

Thanks for looking into this issue.  It's one that will affect any version of the Worldgroup Rlogin client.  Rick Parrish solved it in Gamesrv, so perhaps he has additional details that could help.Sent from my Verizon, Samsung Galaxy smartphone -------- Original message --------From: synacktic notifications@github.com Date: 2/11/21 4:17 PM (GMT-06:00) To: mbbsemu/MBBSEmu MBBSEmu@noreply.github.com Cc: BlaZZZed1980 blazzze@gmail.com, Mention mention@noreply.github.com Subject: Re: [mbbsemu/MBBSEmu] [MBBSEmu] RLogin - Worldgroup 3.x - Sends set of characters on connect (#130) Well boo. I was hoping that was a two birds one stone thing.

—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

TQPS commented 3 years ago

Didn't I hear on the grapevine that it may be attributed to the RLOGIN something something screen size?

enusbaum commented 3 years ago

Going to be testing the PR now that I have a setup where I can replicate the issue:

image

enusbaum commented 3 years ago

@BlaZZZed1980 PR was just merged into main. Try the latest with adding the new additional config option Rlogin.Compatibility and set its value to WG3NT. This should eliminate the extra characters that come in from the Rlogin client shipped with WG3NT.

https://www.mbbsemu.com/Downloads/master/030921

TQPS commented 3 years ago

Chatting with Blaz in WorldLink about this. Confirmed your latest fix works great!

![image](https://user-images.githubusercontent.com/66294837/110553119-98915d00-818c-11eb-8ca4-8d0095feb1c3.png)

Thank you so much!!1!1one