-
Notifications
You must be signed in to change notification settings - Fork 8k
mgmt: mcumgr: transport: udp: Add support for DTLS #97526
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
base: main
Are you sure you want to change the base?
Conversation
Adds support for using DTLS for the transport Signed-off-by: Jamie McCrae <[email protected]>
edad48f
to
3e3ecde
Compare
3e3ecde
to
52f298e
Compare
Adds a sample which demonstrates how to use DTLS over UDP. This copies the certificate files from the echo server sample to use as dummy certificates Signed-off-by: Jamie McCrae <[email protected]>
52f298e
to
6abb7ff
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DTLS socket setup looks fine.
@@ -0,0 +1,38 @@ | |||
# Enable the UDP DTLS MCUmgr transport. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this file be called overlay-udp-dtls.conf
to follow the nomenclature in the sample (not sure if there's a strict rule about that though)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overlay refers to dts overlay files, not Kconfig fragments, but also the wrong overlay-* prefix was removed recently from those files: https://github.com/zephyrproject-rtos/zephyr/tree/6abb7ff76c8609eb1f5dbedb0972855eca26778b/samples/subsys/mgmt/mcumgr/smp_svr
Includes sample update showing how to use feature
Compliance failure is expected, dummy certificates are included in the sample (these are copied directly from the echo server sample)
Fixes #91754
@SynchronicIT please test