Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WOTLK] RA omitting last character of command #877

Closed
antiroot opened this issue Mar 16, 2016 · 2 comments
Closed

[WOTLK] RA omitting last character of command #877

antiroot opened this issue Mar 16, 2016 · 2 comments
Assignees

Comments

@antiroot
Copy link

Current behavior:
When issuing commands via RA using telnet the last character of any line of text is omitted, including entering the username and password. However adding a space or another character following the desired input will then simply ignore the extra character and the input will be accepted.
Expected behavior:

Steps to reproduce:

  1. Enable RA
  2. telnet to RA socket
  3. enter anything that requires the full text (usrname/password/quit)

Client version:
3.3.5a
Commit hash:
a6b0038dffb962c981aa177e306d6242afcecd78

Operating system:
FreeBSD 10.1-RELEASE-p10

It seems that commit f365ffd3cb64f20b6670c70165cc60bd6aff6a69 rewrote parts of RA and the file src/mangosd/RASocket.cpp on line 91 has "newLine - 1" removing the minus one resolves the issue, but logically the code looks like the minus one should be there since it's to strip the newline character when the user presses enter.

@antiroot antiroot changed the title [WOTLK] RA ommiting last character of command [WOTLK] RA omitting last character of command Mar 16, 2016
@emeric254
Copy link

Same behaviour, running on Debian.
The last character send is removed (the one before '\n').
Maybe it's a normal things for Windows system assuming they add an '\r' at this place ...

@namreeb namreeb self-assigned this Jun 22, 2016
@mityada
Copy link

mityada commented Aug 15, 2016

This issue presents in all mangos cores. It is solved by cmangos/mangos-classic#185. It's in mangos-classic, but the code is exactly the same, so it can be easily ported to both mangos-tbc and mangos-wotlk.

Phatcat pushed a commit to cmangos/mangos-classic that referenced this issue Sep 10, 2016
Fixes the truncation of the last input character
Fixes the processing of several commands arrived in the same packet

closes #185
refs cmangos/issues#877
Phatcat pushed a commit to cmangos/mangos-tbc that referenced this issue Sep 10, 2016
Fixes the truncation of the last input character
Fixes the processing of several commands arrived in the same packet

refs cmangos/mangos-classic#185
refs cmangos/issues#877
Fabi pushed a commit to cmangos/mangos-cata that referenced this issue Jan 4, 2017
Fixes the truncation of the last input character
Fixes the processing of several commands arrived in the same packet

refs cmangos/mangos-classic#185
closes cmangos/issues#877
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants