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

Implement Telnet STATUS Option Support #132

Open
wants to merge 2 commits into
base: feat/telnet-ttype
Choose a base branch
from

Conversation

mystiker
Copy link
Collaborator

@mystiker mystiker commented Nov 4, 2024

Description

This pull request introduces support for the Telnet STATUS option, allowing the client to request and receive a report on the current state of negotiated Telnet options with the server. The STATUS option enhances debugging and diagnostics by providing visibility into the active Telnet session configuration. This implementation is based on RFC 859.

Key Changes

  • Added handleStatusOption function: Manages the STATUS option negotiation by handling the DO, DONT, WILL, and WONT commands related to the STATUS option.
  • Subnegotiation support: Upon receiving WILL STATUS from the server, the client sends a STATUS SEND command, prompting the server to respond with its current configuration of active options.
  • Enhanced option handling via dedicated handler functions for each negotiation command, including handleStatusDo, handleStatusDont, handleStatusWill, and handleStatusWont.

Acceptance Criteria

  • The client can initiate a status request and accurately receive a response.
  • The response is correctly parsed, allowing the client to reflect the active Telnet options.
  • Status information is available for debugging and display purposes, helping maintain connection consistency.

Fixes #130

@mystiker mystiker linked an issue Nov 4, 2024 that may be closed by this pull request
@mystiker mystiker linked an issue Nov 4, 2024 that may be closed by this pull request
@mystiker mystiker self-assigned this Nov 4, 2024
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.

Implement Telnet STATUS Option
1 participant