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

piggyback acks in message not supported #42

Open
hemna opened this issue Jan 21, 2021 · 6 comments
Open

piggyback acks in message not supported #42

hemna opened this issue Jan 21, 2021 · 6 comments

Comments

@hemna
Copy link
Collaborator

hemna commented Jan 21, 2021

There is a bug in the aprslib parsing library that doesn't handle parsing out the acks in line number format here:
rossengeorgiev/aprs-python#58

This means aprsd will fail to properly parse out messages/commands when a user sends a message with {WW}AA format

└─[$] <> cat test-2.py
import aprslib
import pprint
msg = "KO4KWC-6>APRS,TCPIP*,qAC,T2MCI::KO4KWC-14:weather ve3kbr {22}AA"
packet = aprslib.parse(msg)
pprint.pprint(packet)

results in

└─[$] <> python test-2.py
{'addresse': 'KO4KWC-14',
 'format': 'message',
 'from': 'KO4KWC-6',
 'message_text': 'weather ve3kbr {22}AA',
 'path': ['TCPIP*', 'qAC', 'T2MCI'],
 'raw': 'KO4KWC-6>APRS,TCPIP*,qAC,T2MCI::KO4KWC-14:weather ve3kbr {22}AA',
 'to': 'APRS',
 'via': 'T2MCI'}
@hemna
Copy link
Collaborator Author

hemna commented Jan 21, 2021

aprsd doesn't support the piggyback ack as a result.

@ldeffenb
Copy link

For additional information on where {xx}yy comes from, read: http://www.aprs.org/aprs11/replyacks.txt

@hemna
Copy link
Collaborator Author

hemna commented Jan 12, 2022

currently APRSD is not embedding acks in reply messages.

@ldeffenb
Copy link

ldeffenb commented Jan 12, 2022

True, but it is also not acking Reply-Ack-compatible clients correctly. Consider:

2022-01-12 12:50:58 EST: KJ4ERJ>APWW11,TCPIP*,qAC,SECOND::REPEAT   :n 2m{HP}
2022-01-12 12:50:58 EST: REPEAT>APZ100,TCPIP*,qAC,T2ALBERTA::KJ4ERJ   :ackHP
2022-01-12 12:50:59 EST: REPEAT>APZ100,TCPIP*,qAC,T2ALBERTA::KJ4ERJ   :K4EOC 146.8950- Toff 0.7mi W{379
2022-01-12 12:51:00 EST: KJ4ERJ>APWW11,TCPIP*,qAC,SECOND::REPEAT   :ack379

2022-01-12 12:51:32 EST: KJ4ERJ>APWW11,TCPIP*,qAC,SECOND::REPEAT   :n 2m{HP}
2022-01-12 12:51:32 EST: REPEAT>APZ100,TCPIP*,qAC,T2ALBERTA::KJ4ERJ   :ackHP
2022-01-12 12:51:32 EST: REPEAT>APZ100,TCPIP*,qAC,T2ALBERTA::KJ4ERJ   :K4EOC 146.8950- Toff 0.7mi W{380
2022-01-12 12:51:33 EST: KJ4ERJ>APWW11,TCPIP*,qAC,SECOND::REPEAT   :ack380

2022-01-12 12:52:04 EST: KJ4ERJ>APWW11,TCPIP*,qAC,SECOND::REPEAT   :n 2m{HP}
2022-01-12 12:52:04 EST: REPEAT>APZ100,TCPIP*,qAC,T2ALBERTA::KJ4ERJ   :ackHP
2022-01-12 12:52:04 EST: REPEAT>APZ100,TCPIP*,qAC,T2ALBERTA::KJ4ERJ   :K4EOC 146.8950- Toff 0.7mi W{381
2022-01-12 12:52:05 EST: KJ4ERJ>APWW11,TCPIP*,qAC,SECOND::REPEAT   :ack381

(Just for the audit-trail)

@hemna
Copy link
Collaborator Author

hemna commented Jan 12, 2022

Here is a current example exchange

today at 12:53:43 PM Received Message _______________                                              
today at 12:53:43 PM   Raw     : KJ4ERJ>APWW11,TCPIP*,qAC,SECOND::REPEAT   :n 5 2m{HQ}
today at 12:53:43 PM    From    : KJ4ERJ                                                             
today at 12:53:43 PM    Message : n 5 2m                                                             
today at 12:53:43 PM    Msg #   : HQ                                                                 
today at 12:53:43 PM  Received Message _______________ Complete

today at 12:53:43 PM  Sending ack _______________                                                    
today at 12:53:43 PM  Raw     : REPEAT>APZ100::KJ4ERJ   :ackHQ                                     
today at 12:53:43 PM  To      : KJ4ERJ                                                             
today at 12:53:43 PM   Ack     : HQ                                                                 
today at 12:53:43 PM  Sending ack _______________ Complete

today at 12:53:44 PM  Sending Message _______________                                                 
today at 12:53:44 PM    Raw     : REPEAT>APZ100::KJ4ERJ   :K4EOC 146.8950- Toff 0.7mi W{383           
today at 12:53:44 PM    To      : KJ4ERJ                                                              
today at 12:53:44 PM    Message : K4EOC 146.8950- Toff 0.7mi W                                        
today at 12:53:44 PM    Msg #   : 383                                                                 
today at 12:53:44 PM  Sending Message _______________ Complete

@hemna
Copy link
Collaborator Author

hemna commented Apr 12, 2024

aprslib supports parsing out piggyback acks now. APRSD doesn't support sending them at the moment.

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

2 participants