https://gitlab.synchro.net/sbbs/sbbs/-/commit/1c4c3c5bdeddae857b931ea0
Modified Files:
src/sexpots/sexpots.c
Log Message:
Resolve GCC warnings.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/sbbs/sbbs/-/commit/f32870268764f23e33afad97
Modified Files:
src/sexpots/sexpots.c
Log Message:
Update Id keyword. Not sure why.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/sbbs/sbbs/-/commit/26f06b3b0396263d4a7cbbab
Modified Files:
src/sexpots/sexpots.c
Log Message:
In Windows build, we need/use that NAME argument in the usage output.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/6fde2a0e4c11b5376ecac3f6
Modified Files:
src/sexpots/sexpots.c
Log Message:
Add a -syslog option for *nix builds to force log output to syslog

for Nelgin
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/c6fde03fbbdbd8efc9d181eb
Modified Files:
src/sexpots/sexpots.c
Log Message:
Fix issue that prevent "-syslog" option from being used (on *nix)

Fixed a couple sprintf() buffer overflow warnings/issues.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/4c33738ef1c22dd74a5122ac
Modified Files:
src/sexpots/sexpots.c
Log Message:
Optionally send a prompt to connected user

Wait up to 10 seconds for a single-char reply to the prompt and if a non-control character was received, use the [TCP:x] (x is the character sent by the user) from the sexpots.ini for the Host and Port values to connect to.

An experimental feature for Nelgin.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/9798f30aeea0432edf99b76d
Modified Files:
src/sexpots/sexpots.c
Log Message:
Make PromptTimeout configurable (default to 0/infinite)

Set to number of seconds to timeout, if desired.
Log the received character in response to the prompt.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/b8dcc1e8e52f5f877134bb5f
Modified Files:
src/sexpots/sexpots.c
Log Message:
Optional clear-screen before sending banner

Set CLS=true in the root section of sexpots.ini to enable. Sends a ^L ASCII 12 0x0c FF CS whatever you want to call it to the remote terminal before sending the carriage return and copyright banner. For Nelgin.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/3efb480e94cf03396a17dc0a
Modified Files:
src/sexpots/sexpots.c
Log Message:
Change default PromptTimeout to 60 (seconds)

Also log a warning if a control character is received in response to the prompt, or log a notice if there's a timeout waiting for a response to the prompt. Change the non-control character response value to a debug-level log message.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/a355fdf9c745695292036c3e
Modified Files:
src/sexpots/sexpots.c
Log Message:
Re-parse the "TCP" section before sending (optional) Prompt

We need to re-parse the default sexpots.ini [TCP] section for every new session (when using the Prompt feature) or else there's no way to connect to the default host.

Also, telnet to be enabled or disabled (e.g. by setting Telnet=false) in any [TCP] or [TCP:x] section to support a mix of Telnet and non-Telnet hosts.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/947e9258b59e56a22cea0a2e
Modified Files:
src/sexpots/sexpots.c
Log Message:
Actually do bounds checking on timeout.

Fix warning while we're here.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/1da032deaaa38823eebe0d94
Modified Files:
src/sexpots/sexpots.c
Log Message:
Allow one second longer for timeout
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/9187c26df0899951d52897e0
Modified Files:
src/sexpots/sexpots.c
Log Message:
Remove the extra stderr output for debug-level log msgs on *nix

As reported by Nelgin
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/af4a8764c35db996c89bb734
Modified Files:
src/sexpots/sexpots.c
Log Message:
Add optional RingCount and RingInterval [modem] settings

Only used in manual answer mode,
RingCount defaults 1 (answer after the first ring) and RingInterval defaults to 6000 (maximum elapsed milliseconds between rings of the same "call").

This should address issue/feature-request #330 by Nelgin. Give that a try.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/6977a080e9258f6e663f7bb5
Modified Files:
src/sexpots/sexpots.c
Log Message:
Support an optional log identifier to syslog()

Nelgin: "anyway to get sexpots to report itself differently to syslog" [?]
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/869e0ebdf76bbc8417682ec1
Modified Files:
src/sexpots/sexpots.c
Log Message:
Setting IgnoreDCD=true causes wait_for_call() to immediate return

For Nelgin's null modem setup, putty doesn't raise DTR?

<nelgin> sexpots 1492 root 3u CHR 4,64 0t0 90 /dev/ttyS0
<nelgin> Well, it's listening, but when I connect with putty I get nothing. <nelgin> DigitalMan, probably always assume a client is connected since I have my pc wired to my linux box.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/8cf2abe7b7402672e0602167
Modified Files:
src/sexpots/sexpots.c
Log Message:
Parse/apply [COM:x] section upon prompted key press of 'x'

Another hack for Nelgin: this'll allow (in theory) the serial device to switch parity modes (e.g. 8N1 or 7E1) depending on what the user selects (key pressed).

I did not add byte/stop bit size configurations. We may need that.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/32fa13dda8e2c1522d4255f8
Modified Files:
src/sexpots/sexpots.c
Log Message:
Don't call com_setup() unless an alternate [COM:x] section is loaded

Apparently com_setup() (which just calls tcsetattr()) will cause an attached modem (with a caller) to disconnect the caller. So you can't do this with a live modem. Leave the hack in for Nelgin's null modem cable setup, but you can't (apparently) support both 8N1 and 7E1 "callers" with a real modem (not surprisingly, for multiple reasons).
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/93cc268f1865ef6a7af2d983
Modified Files:
src/sexpots/sexpots.c
Log Message:
Add optional debug-level logging of all chars received over COM port/device

Set Debug=true in the [com] section of sexpots.ini to enable this (and you'll need to be logging debug-level message as well). For Nelgin.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/a7b7daf1bc7bcd53c25e35db
Modified Files:
src/sexpots/sexpots.c
Log Message:
Fix printf format typo in previous commit

This should fix issue #509
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/a2034ad01c3627976c3d1bcc
Modified Files:
src/sexpots/sexpots.c
Log Message:
[com] Debug will enable transmit debug log output (in addition to receive)

Log the characters themselves of ASCII values (between 32 and 126) for com debug log messages. Still a separate log message/line for every character
(so very chatty/verbose).

This fixes issue (feature request) #510
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/f8c65bcf598afdbddf969b11
Modified Files:
src/sexpots/sexpots.c
Log Message:
Fixup the copy-pasted resolve_ip() with same fixes from main.cpp/mailsrvr.c

Don't null-deref h_addr_list.
Return in_addr_t instead of u_long.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/54a05b65148085247e667152
Modified Files:
src/sexpots/sexpots.c
Log Message:
Implement write-retry in modem_send()

In attempt to address the "Error 11" (EAGAIN) error theat Nelgin sees when configuring some longer modem init strings on Linux.

This is just a single retry (after a yield) after any modem command char
send failure (for any reason), including the terminating carriage-return.

See issue #662 to details.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

Re: src/sexpots/sexpots.c
By: Rob Swindell (on Debian Linux) to Git commit to main/sbbs/master on Mon Oct 16 2023 17:53:57

https://gitlab.synchro.net/main/sbbs/-/commit/54a05b65148085247e667152 Modified Files:
src/sexpots/sexpots.c
Log Message:
Implement write-retry in modem_send()

In attempt to address the "Error 11" (EAGAIN) error theat Nelgin sees when configuring some longer modem init strings on Linux.
This is just a single retry (after a yield) after any modem command char send failure (for any reason), including the terminating carriage-return.

im guessing that it's the &W he has at the end of the init string. some modems (including mine for some reason) won't send an OK when they get that.

as you can see in a previous message, mine is a lot longer than his but works fine. I think his EAGAIN could be a resource conflict in some form of buffer maybe. ie sending data too fast (not that it should matter).

not knowing his setup i couldnt answer that, but i would put money on it being on a cheap usb/serial adapter :D

charlie

---
■ Synchronet ■ The FBO BBS - IPV4/V6 - bbs.thefbo.us or bbs6.thefbo.us

Re: src/sexpots/sexpots.c
By: Charles Blackburn to Rob Swindell (on Debian L on Tue Oct 17 2023 07:36 am

as you can see in a previous message, mine is a lot longer than his but works fine. I think his EAGAIN could be a resource conflict in some form of buffer maybe. ie sending data too fast (not that it should matter).

not knowing his setup i couldnt answer that, but i would put money on it being on a cheap usb/serial adapter :D

i think it's because some people throw settings that cant be stored with that method in the ram. i wouldn't even use a long init string. keep it simple.

---
■ Synchronet ■ ::: BBSES.info - free BBS services :::

https://gitlab.synchro.net/main/sbbs/-/commit/0647142c0bc18ddcae3cacea
Modified Files:
src/sexpots/sexpots.c
Log Message:
Silence odd warning... I would expect these to be the same type
save const, but apparently they're not. *shrug*
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/05dcd69ab61c73a50f5d2879
Modified Files:
src/sexpots/sexpots.c
Log Message:
On Win64, COM_HANDLE is bigger than a long.

Use strtoll() to parse number (except when _BORLANDC_ of course)
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/bd56646d3a9a53ef955c87c1
Modified Files:
src/sexpots/sexpots.c
Log Message:
Move swap version and platform in banner to accommodate 40 column display

This should now display pretty normally on both 40 and 80 column displays.

"FreeBSD" is 7 chars, so make that the max expected platform name (for now).

Addresses issue #746, request from Nelgin
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/caa6e97941952127d440c31d
Modified Files:
src/sexpots/sexpots.c
Log Message:
Output the Git repo/branch/hash details in execution banner

Not sent in connection banner.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/08ab80e5fda4b2d45d23980b
Modified Files:
src/sexpots/sexpots.c
Log Message:
Fix warning about potential uninitialize variable value use
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/d1c8606c97d83387e3d2e05a
Modified Files:
src/sexpots/sexpots.c
Log Message:
Log an error if calls to send() don't return the expected value

This addresses GCC warnings about not using the return value of send().
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/dd287bf2e64f5eafbb2e138f
Modified Files:
src/sexpots/sexpots.c
Log Message:
Enable signal (e.g. Ctrl-C) handling / ignoring

Gracefully terminate with SIGQUIT, SIGINT, and SIGTERM

Ignore SIGHUP and SIGPIPE.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/e2bcf7223ea3817ac6ae4d9d
Modified Files:
src/sexpots/sexpots.c
Log Message:
If configured COM port byte size is less than 8, strip high 8-n bits on RX

Some USB modems (reportedly, USRobotics USB modem) don't strip the parity bit of data received from modem connections operating in < 8 bit modes
(e.g. 7-E-1), as is normally don't with a modem connected to a UART, so we'll do that stripping (forcing to 0) here, as recommended by Deuce.

Nelgin, if you're doing some manually stripping of the 7th bit bytes received from 7-E-1 connected modems and then sent to your server/BBS over TCP, you shouldn't need to do that now.
---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net