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

pkg/wakaama/client_connection: destroy unused DTLS sessions #21035

Conversation

leandrolanzieri
Copy link
Contributor

@leandrolanzieri leandrolanzieri commented Nov 24, 2024

Contribution description

When a DTLS handshake fails, tinyDTLS keeps the session slot as used, and then it will fail on the next attempt unless the session is destroyed. This PR adds this behaviour to the lwm2m client.

Testing procedure

With examples/lwm2m, you can test by starting the client before the server, making it fail the initial handshake attempt.

Issues/PRs references

None

@leandrolanzieri leandrolanzieri added the Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) label Nov 24, 2024
@github-actions github-actions bot added the Area: pkg Area: External package ports label Nov 24, 2024
@leandrolanzieri leandrolanzieri added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Nov 24, 2024
@riot-ci
Copy link

riot-ci commented Nov 24, 2024

Murdock results

✔️ PASSED

22f7799 pkg/wakaama/client_connection: destroy unused DTLS sessions

Success Failures Total Runtime
10242 0 10246 15m:38s

Artifacts

@Teufelchen1 Teufelchen1 added this pull request to the merge queue Nov 25, 2024
Merged via the queue into RIOT-OS:master with commit 71e6c86 Nov 25, 2024
28 checks passed
@MrKevinWeiss MrKevinWeiss added this to the Release 2025.01 milestone Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants