This repository has been archived by the owner. It is now read-only.

login (and whitelisting) fail on usernames with spaces in them #4364

Closed
RangerRick opened this Issue Sep 12, 2017 · 3 comments

Comments

Projects
None yet
4 participants
@RangerRick

RangerRick commented Sep 12, 2017

Issue description

I set up the latest 1.7dev and while it loads fine, I had a number of issues trying to make a username with a space in it work:

  1. I was unable to log in because I had MineCraftPE connected to my Xbox Live gamertag, which has a space in it (Racoon Fink). Attempting to log in would give me the error:
    [21:19:34] [Server thread/INFO]: Racoon Fink[/XXX.XXX.XXX.XXX:23478] logged out due to Invalid name!
  2. I tried logging out of my MS account and tried connecting to the server again with the same username and it still failed with the same error.
  3. Thinking maybe it was a whitelist issue, I tried adding the user to the whitelist, but it would not accept a username with a space in it, whether or not I quoted it.
whitelist add Racoon Fink
[21:21:30] [Server thread/INFO]: Usage: /whitelist <on|off|list|add|remove|reload>
whitelist add "Racoon Fink"
[21:21:37] [Server thread/INFO]: Usage: /whitelist <on|off|list|add|remove|reload>

(Once I changed my username to one without spaces in it, I was able to connect to the server successfully.)

Steps to reproduce the issue

  1. set your name to contain a space in your MineCraftPE profile
  2. attempt to log in

OS and versions

  • PocketMine-MP: dev install as of yesterday afternoon (server version v1.1.0.55 / 1.7dev-31)
  • PHP: 7.2RC1
  • OS: Ubuntu 16 (inside docker)
@joech4n

This comment has been minimized.

Show comment
Hide comment
@joech4n

joech4n commented Sep 15, 2017

+1

@robske110

This comment has been minimized.

Show comment
Hide comment
@robske110

robske110 Sep 15, 2017

Usernames with spaces are not (yet) allowed in PM. PM names must follow the naming convention of MCPC. Microsoft shitbox allows spaces, and therfore this was implemented into MCPE too.

robske110 commented Sep 15, 2017

Usernames with spaces are not (yet) allowed in PM. PM names must follow the naming convention of MCPC. Microsoft shitbox allows spaces, and therfore this was implemented into MCPE too.

@dktapps

This comment has been minimized.

Show comment
Hide comment
@dktapps

dktapps Dec 31, 2017

Member

This has been implemented in newer versions of PMMP PocketMine-MP. Versions supporting Minecraft Bedrock 1.2+ support this by using quotes around "names with spaces".

Member

dktapps commented Dec 31, 2017

This has been implemented in newer versions of PMMP PocketMine-MP. Versions supporting Minecraft Bedrock 1.2+ support this by using quotes around "names with spaces".

@dktapps dktapps closed this Dec 31, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.