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

Update CRMP implementation to comply with final CRMP spec in CHIP v0.7 #4202

Closed
yufengwangca opened this issue Dec 14, 2020 · 3 comments
Closed
Assignees

Comments

@yufengwangca
Copy link
Contributor

Problem

CRMP spec is finally approved and merged, there are several changes from the original proposal based on Weave WRMP spec.

  1. Standalong ack from Secure Channel protocol is used to replace NULL message from Common protocol
  2. CRMP Delayed Delivery mechanism is dropped in the final version.
  3. CRMP Throttle Flow mechanism is dropped in the final version.

There are also several small adjustments besides the three major ones mentioned above.

Proposed Solution

Update CRMP implementation to comply with final CRMP spec in CHIP v0.7

@yufengwangca
Copy link
Contributor Author

Need to clarify in Spec how the node should behave to address the following scenario.

If we didn't find an existing exchange that matches the message, and no unsolicited message handler registered
to hand this message, how the device should address this scenario.

@woody-apple
Copy link
Contributor

@yufengwangca Can you help clarify and split out the remaining work?

@yufengwangca
Copy link
Contributor Author

All three items listed in this tickets have already been addressed already

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants