search

Ly-sec / PalBot

PalBot - Whitelist, RCON, Server management Palworld Discoord Bot
20 stars 3 forks source link

Bot is showing only the first 10 digits of the steam ID, resulting in a kick caused by a mistmatch #7

Closed AWL-Gaming closed 8 months ago

AWL-Gaming commented 8 months ago

At times the bot will show only the 1st 10 digits of the online player's Steam ID, resulting in them being kicked. Moreover, it'll also add 2 digits at the end of the Steam ID. Both issues will make the ID appear as mismatched while it's not. A temporary workaround is to copy the entire whitelist, keep the first 10 digits and paste them under the full IDs which is not practical. The picture below explains the issue: image

Ly-sec commented 8 months ago

At times the bot will show only the 1st 10 digits of the online player's Steam ID, resulting in them being kicked. Moreover, it'll also add 2 digits at the end of the Steam ID. Both issues will make the ID appear as mismatched while it's not. A temporary workaround is to copy the entire whitelist, keep the first 10 digits and paste them under the full IDs which is not practical. The picture below explains the issue: image

This is very odd, I'll see if I can figure out a fix tomorrow. Might be some parsing issue. I'll let you know tomorrow

AWL-Gaming commented 8 months ago

Last two digits added by the bot are only 10 or 13 from what we've obtained so far: image

Ly-sec commented 8 months ago

Last two digits added by the bot are only 10 or 13 from what we've obtained so far: image

I'll see if I can replicate it but I sadly don't have a server with tons of people (which I assume might be why this is even happening...

AWL-Gaming commented 8 months ago

Last two digits added by the bot are only 10 or 13 from what we've obtained so far: image

I'll see if I can replicate it but I sadly don't have a server with tons of people (which I assume might be why this is even happening...

Original whitelist contains about 70 IDs, which is now doubled because of the 10-digit issue's workaround.

Ly-sec commented 8 months ago

Last two digits added by the bot are only 10 or 13 from what we've obtained so far: image

I'll see if I can replicate it but I sadly don't have a server with tons of people (which I assume might be why this is even happening...

Original whitelist contains about 70 IDs, which is now doubled because of the 10-digit issue's workaround.

Gonna check tomorrow if I can fix it. The more I think about it, I think I had a similar issue with the ids just being scuffed like that. We'll see though.

AWL-Gaming commented 8 months ago

Last two digits added by the bot are only 10 or 13 from what we've obtained so far: image

I'll see if I can replicate it but I sadly don't have a server with tons of people (which I assume might be why this is even happening...

Original whitelist contains about 70 IDs, which is now doubled because of the 10-digit issue's workaround.

Gonna check tomorrow if I can fix it. The more I think about it, I think I had a similar issue with the ids just being scuffed like that. We'll see though.

Thank you, keep me updated when you can. Had to pause the whitelist for now since it was kicking the legit whitelisted players as well.

AlexisD09 commented 8 months ago

I had a similar problem with big numbers. As I remember, I manage them as strings and not as numbers. Node have a length limit for digits.

Ly-sec commented 8 months ago

I had a similar problem with big numbers. As I remember, I manage them as strings and not as numbers. Node have a length limit for digits.

Oh this is actually a good call, let's try that too. Working on a different fix right now but I'll turn the ids into strings too, just in case.

Ly-sec commented 8 months ago

@AWL-Gaming I pushed a new update, please try this and see if it works better now :).

AWL-Gaming commented 8 months ago

image Working perfectly fine now! Tried it on my own ID as well, un-whitelisting then relisting and it didn't touch the other IDs nor did it switch the numbers. Thank you for the quick update and fix!