Skip to content

Commit cf70069

Browse files
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.
1 parent a627077 commit cf70069

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
## Next version
44
- 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.
5+
- 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.
56

67
## Version 1.8.1
78
<time datetime="2024-11-03">November 3, 2024</time>

src/aprs-is.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ sendPacket (const char* const restrict server,
217217
}
218218

219219
/* Authenticate */
220-
sprintf(buffer, "user %s pass %s vers %s/%s\n",
220+
sprintf(buffer, "user %s pass %s vers %s %s\n",
221221
username, password, PACKAGE, VERSION);
222222
#ifdef DEBUG
223223
printf("> %s", buffer);

0 commit comments

Comments
 (0)