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

Fix missing return status for LmHandlerJoin #67

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

xanthio
Copy link

@xanthio xanthio commented Apr 3, 2023

In OTAA mode, the join procedure may fail due to Retransmission Back-off (see https://lora-developers.semtech.com/documentation/tech-papers-and-guides/the-book/joining-and-rejoining).

In this case, Application will never receive any OnJoinRequest callback an so may stall.

This PR intend allow Application to be aware of such a failure to send Join Request and behave in consequence.

@xanthio
Copy link
Author

xanthio commented Apr 3, 2023

Note to reviewer: I'm not a ABP mode user and so may have not been placed at the good position.

@ASELSTM ASELSTM added lorawan Long Range Wide Area Network mw Middleware-related issue or pull-request. labels Apr 18, 2023
@ASELSTM
Copy link
Contributor

ASELSTM commented May 15, 2023

ST Internal Reference: 152150

@ASELSTM ASELSTM added the internal bug tracker Issue confirmed and logged into the internal bug tracking system label May 15, 2023
@ALABSTM ALABSTM added the bug Something isn't working label Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working internal bug tracker Issue confirmed and logged into the internal bug tracking system lorawan Long Range Wide Area Network mw Middleware-related issue or pull-request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants