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

chore(es_extended/server): add missing identifier config #1517

Closed
wants to merge 1 commit into from

Conversation

RrybaN
Copy link
Contributor

@RrybaN RrybaN commented Nov 29, 2024

I noticed that the Config.Identifier was being used in esx_multicharacter but not existing on es_extended config so i added it.

Server.identifierType = ESX.GetConfig().Identifier or GetConvar("sv_lan", "") == "true" and "ip" or "license"

@Mycroft-Studios
Copy link
Member

Using anything other than licence is extremely stupid, and a great way to have people exploit your server.
IP: I dont even need to explain this.. i hope
Discord: can create an alt account within minutes and abuse the system,
Steam: can create an alt account within minutes and abuse the system,
Licence2: not all users have 1

@Mycroft-Studios
Copy link
Member

That config option not existing, is intentional :)

@RrybaN
Copy link
Contributor Author

RrybaN commented Nov 29, 2024

I know that players can create alt accounts, and it's ultimately up to the server owner whether they want to use different licenses. For example, if you get a new PC or reformat your own PC, sometimes your license will change, and of course, you cannot retrieve your character. So, it's better to use Steam or Discord licenses in that case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Recycle Bin
Development

Successfully merging this pull request may close these issues.

2 participants