-
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
[app] Move CASESessionManager to server internal storage #12778
Conversation
6816b08
to
7335eb8
Compare
PR #12778: Size comparison from a37aab5 to 7335eb8 Increases above 0.2%:
Increases (14 builds for efr32, k32w, linux, mbed, p6)
Decreases (6 builds for mbed, p6)
Full report (16 builds for efr32, k32w, linux, mbed, p6)
|
So why this move? Neither the commits nor the PR description explain why we want to do this.... |
Updated PR description. |
7335eb8
to
a303882
Compare
PR #12778: Size comparison from d6dfe03 to 1cc39c1 Increases above 0.2%:
Increases (14 builds for efr32, k32w, linux, p6, qpg, telink)
Decreases (3 builds for p6)
Full report (15 builds for efr32, k32w, linux, p6, qpg, telink)
|
79f05b4
to
b6d0022
Compare
PR #12778: Size comparison from 83aac59 to b6d0022 Increases above 0.2%:
Increases (26 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (7 builds for linux, mbed, p6)
Full report (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
b6d0022
to
157de4f
Compare
PR #12778: Size comparison from f85d6ed to 157de4f Increases above 0.2%:
Increases (26 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (7 builds for linux, mbed, p6)
Full report (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Generally seems ok - I have some concerns about not coupling things too much (we start having god objects for containing all 'managers'), however we should be able to decouple if we find it necessary. |
fast track: PR up for a while now, change is reasonably small. |
Problem
CASESessionManager is now only availabe on linux devices.
For OTA and binding cluster to work, we need to share the
CASESessionManager
with the OTA requestor on all platforms.Changes
Move
CASESessionManager
to the Server class.Test
Tested with OTA requestor & provider.