-
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
Add ACL validation #14756
Add ACL validation #14756
Conversation
- There was some before, but this is more stringent. - Also refactor some of the "IsValid"-like functions. - Update unit tests. - Remove more operational PASE support. Part of project-chip#14460
- Simplify validation function - Add more unit tests
I was hoping to get this done today but still a couple things to wrap up tomorrow morning before I flip the draft/review switch. Thank you for your patience. |
- More test cases - Move some of the IsValid functions into access control - Slight simplification to AccessControl::IsValid
Alright, lots of test code now, validity should be solid. |
PR #14756: Size comparison from b080024 to 24f4bec Increases (29 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (34 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #14756: Size comparison from b080024 to fdc0a51 Increases (30 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (1 build for linux)
Full report (34 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Delegate must outlive the thing holding it.
PR #14756: Size comparison from b080024 to 0b85bae Increases above 0.2%:
Increases (38 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (9 builds for esp32, linux, mbed, p6)
Full report (43 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #14756: Size comparison from 983fedf to 1519d3e Increases (21 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
Full report (24 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
|
PR #14756: Size comparison from f4901af to 7db18dc Increases above 0.2%:
Increases (30 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
Full report (33 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, qpg, telink)
|
Problem
Need more ACL validation. #14460
Change overview
Testing