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.

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

aborted connects via agwpe port don't abort #275

Closed
pe1rrr opened this issue May 22, 2020 · 3 comments
Closed

aborted connects via agwpe port don't abort #275

pe1rrr opened this issue May 22, 2020 · 3 comments

Comments

@pe1rrr
Copy link

pe1rrr commented May 22, 2020

Hello,

When connecting to a station, then having 2nd thoughts and disconnecting (sending 'd' before the remote station has responded) dw (direwolf) continues to attempt connections until it reaches the maximum retry limit.

This means it is possible to detach an agwpe terminal session from dw without it first stopping the SABM polls.

`[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
DCD 0 = 1
DCD 0 = 0
PTT 0 = 0
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
DCD 0 = 1
DCD 0 = 0
PTT 0 = 0
Disconnect from EI2GYB-3 ...
Disconnect from EI2GYB-3 ...
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
PTT 0 = 0
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
DCD 0 = 1

PE1RRR-13 audio level = 0(1/1) 0
[0.1] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
PTT 0 = 0
DCD 0 = 0
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
PTT 0 = 0
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
PTT 0 = 0
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
Disconnect from EI2GYB-3 ...
PTT 0 = 0
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)

PE1RRR-13 audio level = 0(0/0) 305
[0.1] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
PTT 0 = 0
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
PTT 0 = 0
PTT 0 = 1
[0L] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)

PE1RRR-13 audio level = 0(1/1) 30_
[0.1] PE1RRR-13>EI2GYB-3:(SABM cmd, p=1)
PTT 0 = 0
Failed to connect to EI2GYB-3 after 10 tries.`

The double reporting of disconnect state 'Disconnect from EI2GYB-3 ...' also seems a bit dubious.

Let me know if I need to provide anything else.

@jmkristian
Copy link
Contributor

I agree this is annoying. It would be nice to send DISC or DM, or at least stop retrying SABM.

@ghost
Copy link

ghost commented Nov 8, 2020

I agree I would like to send a few commands and also send messages.

@wb2osz
Copy link
Owner

wb2osz commented May 7, 2023

This has been fixed in the dev branch, commit 8e32286.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants