-
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
We should not have a CASESessionManager per DeviceController #16174
Comments
This was referenced Mar 14, 2022
Some specific issues regarding current members of
|
Maybe the answer for now is that |
bzbarsky-apple
added a commit
to bzbarsky-apple/connectedhomeip
that referenced
this issue
Mar 15, 2022
We should have one CASESessionManager, not one per DeviceController. Fixes project-chip#16174
bzbarsky-apple
added a commit
to bzbarsky-apple/connectedhomeip
that referenced
this issue
Mar 15, 2022
We should have one CASESessionManager, not one per DeviceController. Fixes project-chip#16174
bzbarsky-apple
added a commit
to bzbarsky-apple/connectedhomeip
that referenced
this issue
Mar 15, 2022
We should have one CASESessionManager, not one per DeviceController. Fixes project-chip#16174
bzbarsky-apple
added a commit
to bzbarsky-apple/connectedhomeip
that referenced
this issue
Mar 15, 2022
We should have one CASESessionManager, not one per DeviceController. Fixes project-chip#16174
bzbarsky-apple
added a commit
to bzbarsky-apple/connectedhomeip
that referenced
this issue
Mar 15, 2022
We should have one CASESessionManager, not one per DeviceController. Fixes project-chip#16174
bzbarsky-apple
added a commit
that referenced
this issue
Mar 18, 2022
We should have one CASESessionManager, not one per DeviceController. Fixes #16174
ArekBalysNordic
pushed a commit
to ArekBalysNordic/connectedhomeip
that referenced
this issue
Mar 18, 2022
…chip#16233) We should have one CASESessionManager, not one per DeviceController. Fixes project-chip#16174
andrei-menzopol
pushed a commit
to andrei-menzopol/connectedhomeip
that referenced
this issue
Apr 14, 2022
…chip#16233) We should have one CASESessionManager, not one per DeviceController. Fixes project-chip#16174
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
CASESessionManager was changed to be fabric-independent, but we still have one per DeviceController (so one per fabric in the setup where we have DeviceControllers).
This causes a few problems:
Proposed Solution
Move the CASESessionManager into CHIPDeviceControllerSystemState and have DeviceController just point to it, not own it.
The text was updated successfully, but these errors were encountered: