-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Message counter implementation #5389
Conversation
Re-evaluate the storing of Admin ID in SecureSessionHandleconnectedhomeip/src/transport/SecureSessionHandle.h Lines 48 to 55 in b49127b
This comment was generated by todo based on a
|
control message counter is not implemented yetconnectedhomeip/src/transport/SecureSessionMgr.cpp Lines 365 to 375 in b49127b
This comment was generated by todo based on a
|
"initial Session Establishment bootstrap" is under specified, use message counter sync protocol for both group and unicast messagesconnectedhomeip/src/transport/SecureSessionMgr.cpp Lines 369 to 379 in b49127b
This comment was generated by todo based on a
|
"initial Session Establishment bootstrap" is under specified, use message counter sync protocol for both group and unicast messagesconnectedhomeip/src/transport/SecureSessionMgr.cpp Lines 369 to 379 in ddfad10
This comment was generated by todo based on a
|
"initial Session Establishment bootstrap" is under specified, use message counter sync protocol for both groupconnectedhomeip/src/transport/SecureSessionMgr.cpp Lines 372 to 382 in 1704604
This comment was generated by todo based on a
|
"initial Session Establishment bootstrap" is under specified, use message counter sync protocol for both groupconnectedhomeip/src/transport/SecureSessionMgr.cpp Lines 372 to 382 in 023e407
This comment was generated by todo based on a
|
5ea8f29
to
578e166
Compare
00a07bf
to
323565f
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.
Almost there!
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.
Thank you!
The merge conflict still needs resolving.
Co-authored-by: Boris Zbarsky <[email protected]>
Size increase report for "nrfconnect-example-build" from 8185019
Full report output
|
Co-authored-by: Boris Zbarsky <[email protected]>
Size increase report for "esp32-example-build" from 8185019
Full report output
|
Problem
Implement and enable MessageCounter according to spec
Summary of Changes