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

Developement #25

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Developement #25

wants to merge 16 commits into from

Conversation

dad98253
Copy link

I have enhanced the basic rcon client to include a simple rcurses based management console (see new -j command line option). It loops on a 3 second interval displaying the server status. A bar at the top displays local time and a bar at the bottom displays active command keys. Pressing the F1 key at the main (status) page will display a list of current users. Selecting a user offers two options: kick or ban. Pressing the F2 key at the main page offers a list of your map. Select a map to change level to it. Pressing the q key at any page will jump back to the previous page without performing any changes.

I hope you find it useful.

@dad98253
Copy link
Author

rcon-select-map

@butaford
Copy link
Contributor

Wow

@lyn1337
Copy link

lyn1337 commented Jun 27, 2021

Thanks, dad!

@n0la
Copy link
Owner

n0la commented Jun 28, 2021

This is awesome, and it looks great. Your version would be a great tool to run in a tmux session in a remote shell.

The issue is however that this goes a bit beyond the scope of rcon. This tool was meant as a small, light-weight tool for shell scripts. But I am more than glad to add your version to the README.

@n0la
Copy link
Owner

n0la commented Jun 28, 2021

There is only one issue with the code. By moving from AF_UNSPEC to AF_INET, you are removing the ability to connect to IPv6 hosts. You can stringify the address using getnameinfo, which can handle both IPv4 and IPv6, if you pass the ai_addr and ai_addrlen members to it.

@n0la
Copy link
Owner

n0la commented Jun 30, 2021

I have added a link to @dad98253 fork to the README.

@dad98253
Copy link
Author

dad98253 commented Jul 12, 2021 via email

@n0la
Copy link
Owner

n0la commented Jul 13, 2021

Re "tool for shell scripts"...I agree, and it can still be used that way. But, I like the interactive capability because it only needs to authorize once. My typical use is often diagnostic in nature. BYW, I usually connect to my server via ssh and then run rcon from the server. (ie., via localhost)

To be frank, I do not wish to add curses code to a purely CLI interface. But that doesn't mean your tool does not have a place. Different tools for different tasks. Mine is for simply shell scripting, and yours is for those that wish a more interactive tool on the console.

That is why I added your tool to the README. So people who look for a more interactive tool know where to find your fork.

Ooops... yes, I normally disable IPv6 on my computers/networks

localhost is easier to write in IPv6 (::1) ;-) (jk btw.)

I wonder if it might be worth adding some words to the readme/man page about rcon security. What do you think?

On how passwords are sent plain text? That is probably a good idea.

@dad98253
Copy link
Author

dad98253 commented Jul 14, 2021 via email

@dad98253
Copy link
Author

dad98253 commented Jul 14, 2021 via email

@butaford
Copy link
Contributor

butaford commented Jul 29, 2021

Is it possible to add Cyrillic support?
image
Classic queries:
image

In -j (stdout) mode, control does not work. No response to keystrokes. The "q" key only works.

P.S. sorry for my bad english

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.

4 participants