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.

Sign up for GitHub

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

Jump to bottom

Implement AX.25 TNC Send and Receive #154

Merged
merged 14 commits into from
Jul 31, 2023
Merged

Implement AX.25 TNC Send and Receive #154

merged 14 commits into from
Jul 31, 2023

Conversation

CBielstein
Copy link
Owner

Description

Implements AX.25 TNC send and receive via TCP IP. Done as a lump to allow testing with real hardware and direwolf for a TCP connected TNC.

Resolves #144
Resolves #145
Resolves #147

Changes

  • Switches packet decode to be bytes instead of string
  • Implements AX.25 encode
  • Implements AX.25 decode
  • Adds AX.25 encode-decode test
  • Adds ability to send packet via TCP TNC in APRSsharp.exe

Validation

  • Tests pass
  • Manual validation with hardware and direwolf

@CBielstein
Split Packet encode by format.
96d88d5
@CBielstein
First stab at AX.25 decode.
b5c8f60
@CBielstein
Document and correct ssid handling.
9a773a4
@CBielstein
First stab at AX.25 encoding.
555234c
@CBielstein
Updates for proper AX.25 formatting of addresses using shifted bytes
598352c
@CBielstein
Add UnsupportedInfo to support partial decodes.
7135e53
@CBielstein
Encoding updates for AX.25 as subset of HDLC. Allow KISS TNC to handl…
75a1991 
…e FCS, flags, etc.
@CBielstein
Refactor Packet constructors to remove unnecessary conversions.
f85bfcd
@CBielstein
Refactor Packet.Encode to reduce encoding round trips. Fixes for AX.2…
514fca2 
…5 encoding.
@CBielstein
Ditch UTF-8 everywhere in favor of ASCII encoding.
ccb8422
@CBielstein
Working transmit! Send bytes over the air! Move to sending bytes inst…
68fc35c 
…ead of string, allow specifying destination in Packet constructor.
@CBielstein
Remove unused Format enum.
4f056ee
@CBielstein
Add roundtrip AX.25 encode/decode test.
8d4c21e
@CBielstein
Trim callsign from launch settings.
becd991
@CBielstein CBielstein added the enhancement New feature or request label Jul 31, 2023
@CBielstein CBielstein self-assigned this Jul 31, 2023
@CBielstein CBielstein merged commit 36965b2 into main Jul 31, 2023
3 checks passed
@CBielstein CBielstein deleted the ax25-rebase branch July 31, 2023 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eddywine eddywine Awaiting requested review from eddywine eddywine is a code owner

Assignees

@CBielstein CBielstein

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

APRSsharp CLI can send via TNC Encode of AX.25 Packets Decode of AX.25 Packet Format via KISS
1 participant
@CBielstein