-
Notifications
You must be signed in to change notification settings - Fork 309
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
Implement rejoin flow #8
Comments
Sentry issue: THE-THINGS-STACK-7P |
@rvolosatovs this has been raised again recently in the Sentry issue above, are you picking it up? Can we move it from Backlog? |
Let's keep it here until we start seeing stable LoRaWAN 1.1 stacks that need this. |
Hey Guys, any plans of implementing this anytime soon ? |
Chiming in to say that RadioLib is ready for this! (Happy to help testing) |
LoRaWAN 1.1 has close-to-zero adoption at the moment. With LoRaWAN 1.2 coming, we will definitely implement this. Assigning @halimi. The publication of LoRaWAN 1.2 is delayed for technical reasons and won't be this year. We will have first class support for LoRaWAN 1.2 as soon as it is there. Do you guys have a need to use rejoin with LoRaWAN 1.1 at the moment? Or is it just compliance testing? |
@johanstokking do you guys have any stats on how much the different LW versions are used? Very interested to see how many devices would use 1.1. I don't have a use for it myself nor has there been an issue raised yet, but as the placeholders are there in RadioLib, I thought I'd offer a testing ground! |
It's non-existing, really. It's far less than 1%.
Thanks for that, we'll make use of it for LoRaWAN 1.2 testing. |
Summary:
Rejoin flow should be supported
Why do we need this?
To be compliant with LoRaWAN1.1 spec
What is already there? What do you see now?
No rejoin flow handling
What is missing? What do you want to see?
Rejoin flow handling
How do you propose to implement this?
It should be in big part similar to
Join-Request
handlingOriginal issue: https://github.com/TheThingsIndustries/lorawan-stack/issues/557 by @rvolosatovs
The text was updated successfully, but these errors were encountered: