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.

Sign up for GitHub

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

Jump to bottom

examples/dtls-sock: make use of helper functions #20125

Merged
merged 2 commits into from Jan 22, 2024
Merged

examples/dtls-sock: make use of helper functions #20125

merged 2 commits into from Jan 22, 2024

Conversation

benpicco
Copy link
Contributor

Contribution description

We have helper functions for parsing the destination address and establishing a DTLS connection now, so don't open-code it.

Testing procedure

The app still works as described

main(): This is RIOT! (Version: 2024.01-devel-246-gde89f7-examples/dtls-sock-cleanup)
DTLS sock example application
All up, running the shell now
> dtlsc fe80::581a:98ff:fe23:2d6c test
Connection to server successful
Sent DTLS message
Received 4 bytes: "test"
Terminating

Issues/PRs references

@github-actions github-actions bot added the Area: examples Area: Example Applications label Nov 30, 2023
@benpicco benpicco added Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Nov 30, 2023
@riot-ci
Copy link

riot-ci commented Nov 30, 2023 •
edited

Murdock results

✔️ PASSED

f723f13 examples/dtls-sock: re-use handshake buffer

Success Failures Total Runtime
17 0 17 01m:22s

Artifacts

@benpicco benpicco requested a review from maribu December 28, 2023 15:04
@benpicco benpicco added this pull request to the merge queue Jan 22, 2024
Merged via the queue into RIOT-OS:master with commit a384707 Jan 22, 2024
25 checks passed
@MrKevinWeiss MrKevinWeiss added this to the Release 2024.01 milestone Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maribu maribu maribu approved these changes

@chrysn chrysn Awaiting requested review from chrysn

@miri64 miri64 Awaiting requested review from miri64

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri

Assignees
No one assigned
Labels
Area: examples Area: Example Applications CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Milestone
Release 2024.01
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@benpicco @riot-ci @maribu @MrKevinWeiss