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

Error in database internal command #9

Closed
casmssm opened this issue Oct 2, 2023 · 9 comments
Closed

Error in database internal command #9

casmssm opened this issue Oct 2, 2023 · 9 comments

Comments

@casmssm
Copy link

casmssm commented Oct 2, 2023

Hi, first congratulation for this deployment.

I deployed this solution using Docker with 2 containers MySQL + Server.

I've get this error on login in an account with character builded. The character not show and I see a create button, not it name.
Another question, I see the database and not found a bind between character and account, missing something?

==> /home/server/cabal/log/masterserver.log <==
2023-10-02 07:16:50.627 [ERROR] [DATABASE] Error 1064 (42000): You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ', face, color, hair, aura, gender, show_helmet FROM characters WHERE id = ?' at line 1

Another issue with this deployment type is after login, the Cabal client try to connect in the internal ip address of GameServer and not the address set in the configuration. I changed the container interface to be the same of the host and works.

@ubis
Copy link
Owner

ubis commented Oct 2, 2023

Hi,

I assume you're using the latest build? Could you share your characters table?

As for

the Cabal client try to connect in the internal ip address of GameServer and not the address set in the configuration.

This is because of d164582 which allows to connect locally and remotely. I might make this configurable.

@artem-alekseev
Copy link
Collaborator

artem-alekseev commented Dec 21, 2023

@ubis the word "rank" it's reserved word in sql, change the column name

@ubis
Copy link
Owner

ubis commented Dec 22, 2023

@artem-alekseev can you also reproduce this?

It seems rank is reserved keyword since MySQL 8.0.2, probably because of RANK() function. I guess adding backticks might help.

Though it's weird, I don't get this on my dev server on macOS with MariaDB 11.2.2 nor on my test server on FreeBSD with MariaDB 10.11.5

@artem-alekseev
Copy link
Collaborator

@ubis yep u right, it's error only in mysql, mariadb don't have this problem

@ubis
Copy link
Owner

ubis commented Dec 22, 2023

Aha, that's the first difference I've heard between MariaDB and MySQL. Any reason you guys are using MySQL over MariaDB? IMO MySQL is slowly being killed by Oracle.

Anyway, we should still fix this. @artem-alekseev if you have MySQL installed, could you check with backticks?

@artem-alekseev
Copy link
Collaborator

artem-alekseev commented Dec 22, 2023

I'll take a look tonight and check it out and send you a PR.

@ubis ubis closed this as completed in 28e99bd Dec 22, 2023
@ubis ubis reopened this Dec 22, 2023
@ubis
Copy link
Owner

ubis commented Dec 22, 2023

Another issue with this deployment type is after login, the Cabal client try to connect in the internal ip address of GameServer and not the address set in the configuration. I changed the container interface to be the same of the host and works.

Fixed this with 28e99bd. Now this behaviour is disabled by default, you can still enable it with use_local_ip=1 in gameserver conf.

@artem-alekseev
Copy link
Collaborator

artem-alekseev commented Dec 22, 2023

@ubis the word "rank" it's reserved word in sql, change the column name

@ubis Fixed in #10

I would like to help you, do some simple things, can you tell me where to start?

@ubis
Copy link
Owner

ubis commented Dec 22, 2023

@casmssm everything should be fixed in fb5bbcc or newer revision.

@artem-alekseev we should chat about this in Discord or so, let's not pollute this issue.

@ubis ubis closed this as completed Dec 22, 2023
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

3 participants