Skip to content

Repentogon socket #517

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

Closed

Conversation

elliottcarlson
Copy link

No description provided.

@jsgnextortex
Copy link
Collaborator

Allowing socket without luadebug is far too risky for the modding community and while allowing luadebug, all the api restrictions go out of the window anyway, by that point you can run any lua you want and do anything you want, even without repentogon....for this reason, luadebug, and its functionalities, were never really a focus for the project.
In other words, while the whole point of repentogon is to extend the api without needing to use luadebug, the socket library is out of scope and far too much of a security nightmare to consider....so this PR is unlikely to be merged, if I need to be honest. BUT, take in mind that REPENTOGON allows dll mods and you could tackle it that way, as an example theres catinsurance's dicord rich pressence mod, if you want to take a look:
https://github.com/catinsurance/Repentogon-Discord-Rich-Presence

@elliottcarlson
Copy link
Author

Had an itchy trigger finger on this PR -- #518 has a PR description. Will close this one with comment.

@jsgnextortex This does not remove the need for --luadebug - it just fixes it when --luadebug is enabled.

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

Successfully merging this pull request may close these issues.

2 participants