From cf70069e30456859fc83414492f3b9f3f780946b Mon Sep 17 00:00:00 2001 From: Colin Cogle Date: Sun, 3 Nov 2024 22:35:53 -0500 Subject: [PATCH] Fix bug in APRS-IS server login. The `vers` field in the APRS-IS server login is optional, but if you choose to use it (`aprs-weather-submit` does), then you need to send the app name and version, separated by a space. I was using a slash. My bad. This commit fixes it. That being said, I've never had a problem with this app before, so I'm going to guess that this is a mostly cosmetic change. --- CHANGELOG.md | 1 + src/aprs-is.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 9e6e309..252a887 100755 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,6 +2,7 @@ ## Next version - Fixed a bug where v1.8.1 would not handle timeouts correctly using the Win32 API. The changes are queued for the next version, though I've already included them in the released binaries. +- Fixed a bug in APRS server negotiation. The `vers` field expects the app name and version, separated by a space; however, I was separating it with a forward slash instead. ## Version 1.8.1 diff --git a/src/aprs-is.c b/src/aprs-is.c index 1ff57db..c5783da 100755 --- a/src/aprs-is.c +++ b/src/aprs-is.c @@ -217,7 +217,7 @@ sendPacket (const char* const restrict server, } /* Authenticate */ - sprintf(buffer, "user %s pass %s vers %s/%s\n", + sprintf(buffer, "user %s pass %s vers %s %s\n", username, password, PACKAGE, VERSION); #ifdef DEBUG printf("> %s", buffer);