Skip to content
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 extension support to access control #10252

Closed
mlepage-google opened this issue Oct 5, 2021 · 3 comments
Closed

Add extension support to access control #10252

mlepage-google opened this issue Oct 5, 2021 · 3 comments
Assignees
Labels
acl Access Control feature V1.0

Comments

@mlepage-google
Copy link
Contributor

mlepage-google commented Oct 5, 2021

We'll do this last, since it's basically a copy/paste of ACL attribute support to handle the similar-but-simpler extension attribute.

Includes:

  • extension support in the module
  • extension persistence
  • read extension in the cluster
  • write extension in the cluster
  • event notification in the cluster
  • whatever else ACL does, check if extension needs to do it as well
@mlepage-google
Copy link
Contributor Author

Spec PR reduces extension size from 254 to 128, so also do that change for SDK as part of this work.
https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/5079

mlepage-google added a commit to mlepage-google/connectedhomeip that referenced this issue Apr 4, 2022
Includes attribute. Does not include events, or fabric removal.

Part of issue project-chip#10252
@mlepage-google
Copy link
Contributor Author

Note PR #17008 adds extension attribute support with events, but doesn't handle the fabric removal part which is also necessary. This issue will stay open until that part is done as well.

mlepage-google added a commit to mlepage-google/connectedhomeip that referenced this issue Apr 5, 2022
Extension attribute needs its fabric scoped data removed
when the fabric is removed.

Part of issue project-chip#10252
woody-apple pushed a commit that referenced this issue Apr 5, 2022
* Add extension attribute to access control cluster

Includes attribute. Does not include events, or fabric removal.

Part of issue #10252

* Remove temp log

* Add extension event support

Also change the keys for storing extensions based on review feedback.

* Relax VerifyOrDie

Use CHIP_ERROR_INCORRECT_STATE instead.

* Update src/app/clusters/access-control-server/access-control-server.cpp

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Change some error codes

For invalid octstr length and invalid list length, instead use
constraint error.

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
woody-apple pushed a commit to mlepage-google/connectedhomeip that referenced this issue Apr 5, 2022
Extension attribute needs its fabric scoped data removed
when the fabric is removed.

Part of issue project-chip#10252
mlepage-google added a commit that referenced this issue Apr 5, 2022
Extension attribute needs its fabric scoped data removed
when the fabric is removed.

Part of issue #10252
@mlepage-google
Copy link
Contributor Author

PR #17032 added remove fabric support. Therefore this feature is done.

chencheung pushed a commit to chencheung/connectedhomeip that referenced this issue Apr 6, 2022
* Add extension attribute to access control cluster

Includes attribute. Does not include events, or fabric removal.

Part of issue project-chip#10252

* Remove temp log

* Add extension event support

Also change the keys for storing extensions based on review feedback.

* Relax VerifyOrDie

Use CHIP_ERROR_INCORRECT_STATE instead.

* Update src/app/clusters/access-control-server/access-control-server.cpp

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Change some error codes

For invalid octstr length and invalid list length, instead use
constraint error.

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
chencheung pushed a commit to chencheung/connectedhomeip that referenced this issue Apr 6, 2022
Extension attribute needs its fabric scoped data removed
when the fabric is removed.

Part of issue project-chip#10252
chencheung pushed a commit to chencheung/connectedhomeip that referenced this issue Apr 6, 2022
* Add extension attribute to access control cluster

Includes attribute. Does not include events, or fabric removal.

Part of issue project-chip#10252

* Remove temp log

* Add extension event support

Also change the keys for storing extensions based on review feedback.

* Relax VerifyOrDie

Use CHIP_ERROR_INCORRECT_STATE instead.

* Update src/app/clusters/access-control-server/access-control-server.cpp

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Change some error codes

For invalid octstr length and invalid list length, instead use
constraint error.

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
chencheung pushed a commit to chencheung/connectedhomeip that referenced this issue Apr 6, 2022
Extension attribute needs its fabric scoped data removed
when the fabric is removed.

Part of issue project-chip#10252
andrei-menzopol pushed a commit to andrei-menzopol/connectedhomeip that referenced this issue Apr 14, 2022
* Add extension attribute to access control cluster

Includes attribute. Does not include events, or fabric removal.

Part of issue project-chip#10252

* Remove temp log

* Add extension event support

Also change the keys for storing extensions based on review feedback.

* Relax VerifyOrDie

Use CHIP_ERROR_INCORRECT_STATE instead.

* Update src/app/clusters/access-control-server/access-control-server.cpp

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Change some error codes

For invalid octstr length and invalid list length, instead use
constraint error.

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
andrei-menzopol pushed a commit to andrei-menzopol/connectedhomeip that referenced this issue Apr 14, 2022
Extension attribute needs its fabric scoped data removed
when the fabric is removed.

Part of issue project-chip#10252
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
acl Access Control feature V1.0
Projects
None yet
Development

No branches or pull requests

4 participants