-
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
Update CASE session to match latest specifications #9904
Conversation
Add support for optional MRP parametersconnectedhomeip/src/protocols/secure_channel/CASESession.cpp Lines 526 to 536 in 017b46f
This comment was generated by todo based on a
|
Set timestamp on the new session, to allow selecting a least-recently-used session for evictionconnectedhomeip/src/protocols/secure_channel/CASESession.cpp Lines 1136 to 1146 in 6dc0d2a
This comment was generated by todo based on a
|
Set timestamp on the new session, to allow selecting a least-recently-used session for evictionconnectedhomeip/src/protocols/secure_channel/CASESession.cpp Lines 1333 to 1343 in 6dc0d2a
This comment was generated by todo based on a
|
4313878
to
a2bbac4
Compare
Size increase report for "gn_qpg-example-build" from 0e52970
Full report output
|
Size increase report for "esp32-example-build" from 0e52970
Full report output
|
@msandstedt , @saurabhst , @LuDuda , do you have any review feedback? |
Problem
The CASE Sigma protocol message definitions have been updated in the spec, and SDK code needs update.
Change overview
Update message definition to match the specification
Also update state machine in lieu of Status Report, and CASE resume messages.
Pending: Add mechanism to trigger CASE resumption, and corresponding tests
Testing
Ran TestCASESession unit tests to make sure the tests continue to work with updated message definitions.