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

[ICD] Implement client-side Unsolicited Message handler for check-in messages #28232

Closed
mkardous-silabs opened this issue Jul 24, 2023 · 1 comment · Fixed by #30731
Closed
Assignees

Comments

@mkardous-silabs
Copy link
Contributor

mkardous-silabs commented Jul 24, 2023

Description

Controllers need to register an UnsolicitedMessageHandler to be able to receive, process check-in message sent by ICDs, and refresh key as needed.

@mkardous-silabs mkardous-silabs added the icd Intermittently Connected Devices label Jul 24, 2023
@github-actions github-actions bot removed the icd Intermittently Connected Devices label Jul 24, 2023
@mkardous-silabs mkardous-silabs added the icd Intermittently Connected Devices label Jul 24, 2023
@mkardous-silabs mkardous-silabs changed the title [ICD] Implement Check-In Unsolicited Message handler [ICD] Implement client-side Unsolicited Message handler for check-in messages Jul 24, 2023
@github-actions github-actions bot removed the icd Intermittently Connected Devices label Jul 24, 2023
@woody-apple woody-apple added this to the 1.3 milestone Jul 25, 2023
@jtung-apple jtung-apple added the icd Intermittently Connected Devices label Aug 7, 2023
@jtung-apple jtung-apple self-assigned this Aug 7, 2023
@thivya-amazon thivya-amazon self-assigned this Sep 26, 2023
@yunhanw-google
Copy link
Contributor

you can consider to take ICDClientStorage.hwhen you init your checkin handler, then implement CHIP_ERROR DefaultICDClientStorage::ProcessCheckInPayload(const ByteSpan & payload, ICDClientInfo & clientInfo)

in this callback function, it would do nouce validation via CheckinMessage::ParseCheckinMessagePayload, then check against counter, if counter is out of range, just ask service or locally in our case to refresh key, then further notify to restart subscription.

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