yunhanw-google

remove read client/handler and write handler when corresponding fabric is removed

5

#### Problem https://github.com/project-chip/connectedhomeip/issues/21084 #### Change overview remove read client, read handler and write handler when corresponding fabric is removed #### Testing add unit testing to validate if read client and...

Fix status reponse in command sender

3

#### Problem Fix status response in command sender This PR is cut from https://github.com/project-chip/connectedhomeip/pull/19356 #### Change overview Then onMessageReceived in CommandSender would send status response message with invalid action when...

A followup: we should just remove `Abort`. It's only used from `InteractionModelEngine::Shutdown`, which could just as easily check `IsFree` and call `Close`. _Originally posted by @bzbarsky-apple in https://github.com/project-chip/connectedhomeip/pull/21204#discussion_r932260482_

```suggestion // Applications may hold references to CommandHandler instances for async command processing. // Therefore we can't forcible destroy CommandHandlers here. Their exchanges will get closed by // the fabric...

Per Slack discussion: the WriteClient constructor actually behaves differently depending on `CONFIG_BUILD_FOR_HOST_UNIT_TEST`, so this test is completely broken if that's ever not defined.... We need to fix things so that's...

I couldn't figure out how this was passing, then realized that the only reason it does is that there `#if CHIP_CONFIG_IM_ENABLE_SCHEMA_CHECK` code in `ReadHandler::ProcessReadRequest` that causes the return value to...

### Reproduction steps ```bash currently we have used std::move vector's memory when execute sendAutoResubscribeRequest, which is not right..and we also need implement OnDeallocatePaths callback when using autoresubscribe in jni. https://github.com/project-chip/connectedhomeip/blob/master/src/controller/java/CHIPDeviceController-JNI.cpp#L1161...

Note, though, that except for proxies the Node id can (and should!) be omitted.

2

Note, though, that except for proxies the Node id can (and should!) be omitted. _Originally posted by @bzbarsky-apple in https://github.com/project-chip/connectedhomeip/pull/11500#discussion_r744038065_

Followup: We'll want to just pass the statusIB to the OnError callback, right? Instead of dropping the cluster-specific status...

1

Followup: We'll want to just pass the statusIB to the OnError callback, right? Instead of dropping the cluster-specific status... _Originally posted by @bzbarsky-apple in https://github.com/project-chip/connectedhomeip/pull/10460#discussion_r729051027_

Clarify the message context life cycle management

4

@bzbarsky-apple If I understand correctly, `ProcessReportData` will trigger the transmission of a StatusResponse, which is the final message in this exchange, resulting in the closure of the exchange automatically. So...