-
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
Added steps to validate CASE Authentication Tags (CAT) #11806
Conversation
PR #11806: Size comparison from 1903401 to 63d1860 Full report (35 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
I thought that even with the move from 32-bit tags to the split 16-bit tag ID and 16-bit tag version we still had CAT1 and CAT2 RDNs. Does this PR correctly support that? |
Hi Vijay, just seeing this now (was sick yesterday). PR #11550 is now merged, which does provide a full implementation of Access Control module (storing entries etc.) using full interface (iterating entries etc.). The SubjectDescriptor has changed slightly (since we can put everything into a NodeId), and the test code is entirely rewritten. Also I know Evgeni is working on adding CATs into the session, its test code (including certs), and then to the subject descriptor (e.g. PR #11697). Therefore this will require coordination and rework, my apologies on that. I'll reach out to you on slack. |
Hi Marc, I started working on #10243 as it wasn't assigned to anyone. Sorry, I didn't know it's actively being worked on. Let us discuss this in Slack and see how to collaborate. |
Abandoning this PR as Evgeni is working on it. |
Problem
Change overview
Testing