https://gitlab.synchro.net/main/sbbs/-/commit/b846192f8b436fac248fd9d9f5f638adba6b35ce#note_967

Out of curiosity, I looked up how BinkD does the protocol version parsing:
```
#define PRTCLNAME "binkp"
...
if (!memcmp (s, "VER ", 4) &&
(a = strstr (s, PRTCLNAME "/")) != 0 &&
(b = strstr (a, ".")) != 0)
{
state->major = atoi (a + 6);
state->minor = atoi (b + 1);
Log (6, "remote uses " PRTCLNAME " v.%i.%i", state->major, state->minor); ```
So... that would match "binkp/1.1" *anywhere* in the VER string (even in parenthesis), likely explaining why IRex never had a problem with BinkD in this regard.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

Re: Comment on Commit: Just parse VER using /^VER (.*) ([^ ]*?)$/
By: Rob Swindell to Synchronet note on Thu Nov 26 2020 12:21 pm

So... that would match "binkp/1.1" *anywhere* in the VER string (even in parenthesis), likely explaining why IRex never had a problem with BinkD in this regard.

I had one more dropped connection this morning but already had debugging off. It did print deuce's warning msg on assuming binkp/1.1 so the matching is working but other then that, I don't know why it dropped. I haven't noticed any additional connection drops since, although I'm catching an earful on the MSGID now which I sent them to the wiki.

-altere

---
þ Synchronet þ Athelstan BBS þ athelstan.org þ telnet:23 / ssh:2222