Skip to content

feat: allow execution of raw RCON commands via WebSocket clients#388

Open
anklav24 wants to merge 2 commits intoTeam-Silver-Sphere:masterfrom
anklav24:allow-ws-rcon-execute
Open

feat: allow execution of raw RCON commands via WebSocket clients#388
anklav24 wants to merge 2 commits intoTeam-Silver-Sphere:masterfrom
anklav24:allow-ws-rcon-execute

Conversation

@anklav24
Copy link

@anklav24 anklav24 commented Nov 5, 2024

No description provided.

@werewolfboy13 werewolfboy13 added core bug Bug related to the core SquadJS API patch Patch Change labels Nov 5, 2024
werewolfboy13
werewolfboy13 reviewed Nov 8, 2024
View reviewed changes
Copy link
Collaborator

@werewolfboy13 werewolfboy13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Thomas-Smyth you good with this? If the changes won't break anything.

@watchful-eyes
Copy link

watchful-eyes commented Nov 10, 2024

@Thomas-Smyth you good with this? If the changes won't break anything.

Related PR:
#205

@anklav24
Copy link
Author

anklav24 commented Nov 10, 2024

@Thomas-Smyth you good with this? If the changes won't break anything.

Related PR: #205

Got it! I actually like this option with super even more. The issue here is that the Socket IO plugin does not expose to use methods from the mail class, such as execute.

@JiaruiGe
Copy link

JiaruiGe commented Jul 18, 2025

May I ask if it is possible to use rcon to send native commands after connecting to socketio in an external program after this modification, such as AdminForceTeamChange id

@lbzepoqo
Copy link
Contributor

lbzepoqo commented Jul 18, 2025

May I ask if it is possible to use rcon to send native commands after connecting to socketio in an external program after this modification, such as AdminForceTeamChange id

With this pull request, you can

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

Reviewers

@werewolfboy13 werewolfboy13 werewolfboy13 left review comments

@Thomas-Smyth Thomas-Smyth Awaiting requested review from Thomas-Smyth

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

core bug Bug related to the core SquadJS API patch Patch Change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@anklav24 @watchful-eyes @JiaruiGe @lbzepoqo @werewolfboy13