[SL-ONLY] Provide prints for hooks for customer on demo application - #1236
Rohan Sahay (rosahay-silabs) wants to merge 1 commit into
Conversation
…atus in MQTT example
|
bugbot review |
PR SummaryLow Risk Overview On first MQTT client init, the demo now registers Reviewed by Cursor Bugbot for commit 1b198be. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 1b198be. Configure here.
|
|
||
| case DeviceEventType::kSecureSessionEstablished: | ||
| ChipLogProgress(DeviceLayer, "MQTT demo: Commissioning Started"); | ||
| break; |
There was a problem hiding this comment.
Wrong event for commissioning started
Medium Severity
kSecureSessionEstablished is logged as commissioning start, but that event fires for every PASE and CASE session. After commissioning, each controller reconnect prints Commissioning Started, which misleads the customer logs this sample is meant to provide.
Reviewed by Cursor Bugbot for commit 1b198be. Configure here.


Summary
As requested by agarwal57 , adding sample of
OnPlatformhandler and printing the events requested by customer on the demo application.Related issues
N/A
Testing
Verified the prints on disconnection and reconnection events, the commissioning events are not seen completely as the service starts post WiFi connectivity.