-
Notifications
You must be signed in to change notification settings - Fork 155
fix(rbac): implement conditional aliases #1847
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
fix(rbac): implement conditional aliases #1847
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
82e33e5
to
72100d7
Compare
13c53a4
to
393ec70
Compare
170725b
to
60aee15
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested this and it works as expected. Looking pretty good to me so far, can you also add some documentation for this new feature?
60aee15
to
3dbd7c0
Compare
56b1a1e
to
b4e75ad
Compare
I updated demo to count changes. |
bbc63d9
to
91ba830
Compare
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
91ba830
to
c7be861
Compare
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
## @janus-idp/backstage-plugin-rbac-common [1.8.2](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06) ### Bug Fixes * **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
## [1.17.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06) ### Features * **bulk-import:** add fields for annotations, labels and spec input ([#1950](#1950)) ([a1b790a](a1b790a)) * **rbac:** nested condition ([#1814](#1814)) ([228f1a9](228f1a9)) ### Bug Fixes * **aap:** revert downgrade of janus-cli ([#2012](#2012)) ([97c9667](97c9667)) * **acr:** revert downgrade of shared-react and janus-cli ([#2009](#2009)) ([8c28891](8c28891)) * **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
## [1.27.1](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06) ### Bug Fixes * **aap:** revert downgrade of janus-cli ([#2012](#2012)) ([97c9667](97c9667)) * **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
## [4.7.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06) ### Features * **bulk-import:** add fields for annotations, labels and spec input ([#1950](#1950)) ([a1b790a](a1b790a)) * **rbac:** nested condition ([#1814](#1814)) ([228f1a9](228f1a9)) ### Bug Fixes * **aap:** revert downgrade of janus-cli ([#2012](#2012)) ([97c9667](97c9667)) * **acr:** revert downgrade of shared-react and janus-cli ([#2009](#2009)) ([8c28891](8c28891)) * **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
* fix(rbac): implement conditional aliases Signed-off-by: Oleksandr Andriienko <[email protected]> * fix(rbac): fix build Signed-off-by: Oleksandr Andriienko <[email protected]> * fix(rbac): rework currentUser alias to cover ownershipEntityRefs Signed-off-by: Oleksandr Andriienko <[email protected]> * fix(rbac): add more unit tests Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): make feature working after backstage update to 1.29.2 Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): fix compilation and tests Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): Support two aliases: currentUser and ownerRefs Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): add doc about conditional policies aliases Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): clean up unit test Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): clean up Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): clean up Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): fix dependencies Signed-off-by: Oleksandr Andriienko <[email protected]> * feat(rbac): update yarn.lock Signed-off-by: Oleksandr Andriienko <[email protected]> --------- Signed-off-by: Oleksandr Andriienko <[email protected]>
## 1.0.0 (2024-08-06) ### Bug Fixes * **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
## @janus-idp/backstage-plugin-rbac-common [1.8.2](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06) ### Bug Fixes * **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
## [1.17.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06) ### Features * **bulk-import:** add fields for annotations, labels and spec input ([janus-idp#1950](janus-idp#1950)) ([a1b790a](janus-idp@a1b790a)) * **rbac:** nested condition ([janus-idp#1814](janus-idp#1814)) ([228f1a9](janus-idp@228f1a9)) ### Bug Fixes * **aap:** revert downgrade of janus-cli ([janus-idp#2012](janus-idp#2012)) ([97c9667](janus-idp@97c9667)) * **acr:** revert downgrade of shared-react and janus-cli ([janus-idp#2009](janus-idp#2009)) ([8c28891](janus-idp@8c28891)) * **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
## [1.27.1](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06) ### Bug Fixes * **aap:** revert downgrade of janus-cli ([janus-idp#2012](janus-idp#2012)) ([97c9667](janus-idp@97c9667)) * **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
## [4.7.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06) ### Features * **bulk-import:** add fields for annotations, labels and spec input ([janus-idp#1950](janus-idp#1950)) ([a1b790a](janus-idp@a1b790a)) * **rbac:** nested condition ([janus-idp#1814](janus-idp#1814)) ([228f1a9](janus-idp@228f1a9)) ### Bug Fixes * **aap:** revert downgrade of janus-cli ([janus-idp#2012](janus-idp#2012)) ([97c9667](janus-idp@97c9667)) * **acr:** revert downgrade of shared-react and janus-cli ([janus-idp#2009](janus-idp#2009)) ([8c28891](janus-idp@8c28891)) * **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
What doest this pull request do:
Implement conditional aliases
What does this pull request fix:
https://issues.redhat.com/browse/RHIDP-1765
Demo
(https://youtu.be/XyAf6K28ISE)