Skip to content

Enhanced status command with CPU, network, and TLS info#507

Open
tsautier wants to merge 1 commit into
drftpd-ng:masterfrom
tsautier:fix/issue-12
Open

Enhanced status command with CPU, network, and TLS info#507
tsautier wants to merge 1 commit into
drftpd-ng:masterfrom
tsautier:fix/issue-12

Conversation

@tsautier

@tsautier tsautier commented Jan 14, 2026

Copy link
Copy Markdown

Problem

The !status / SITE STATUS command lacked detailed information about CPU, networking, and TLS connection status between master and slaves.

Solution

Enhanced the status command with new options showing:

  • CPU: Architecture, cores, and system load average
  • Network: Bind address, port, online/total slaves count
  • Slaves: Per-slave details including TLS protocol, cipher suite, and remote address

Changes Made

  • Modified RemoteSlave.java to capture TLS session info (protocol, cipher suite) on connect
  • Added getTlsProtocol(), getTlsCipherSuite(), getRemoteAddress() getters
  • Added cpu option to ServerStatus.doStatus()
  • Added network option to show master bind info and slave counts
  • Added slaves option showing per-slave TLS details

Fixes #12

@tsautier tsautier changed the title Enhanced status command with CPU, network, and TLS info Fixes #12: Enhanced status command with CPU, network, and TLS info Jan 14, 2026
@tsautier tsautier changed the title Fixes #12: Enhanced status command with CPU, network, and TLS info Enhanced status command with CPU, network, and TLS info Jan 14, 2026
- Added TLS protocol/cipher suite tracking in RemoteSlave.connect()
- New 'cpu' status option showing architecture, cores, and load average
- New 'network' status option showing bind address, port, online/total slaves
- New 'slaves' status option showing per-slave TLS protocol, cipher, and address
@tsautier

Copy link
Copy Markdown
Author

@mvangoor @hrxcodes for review

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.

!status

1 participant