-
Notifications
You must be signed in to change notification settings - Fork 27
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
CORE-20721: Add RBAC Group rest resource #6233
CORE-20721: Add RBAC Group rest resource #6233
Conversation
Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
…group Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
e2441b7
to
5c38681
Compare
Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
gradle.properties
Outdated
@@ -39,7 +39,7 @@ commonsLangVersion = 3.12.0 | |||
commonsTextVersion = 1.10.0 | |||
# Corda API libs revision (change in 4th digit indicates a breaking change) | |||
# Change to 5.3.0.xx-SNAPSHOT to pick up maven local published copy | |||
cordaApiVersion=5.3.0.14-beta+ | |||
cordaApiVersion=5.3.0.14-alpha-1719505397377 |
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.
I will revert this once corda/corda-api#1666 has been merged
Jenkins build for PR 6233 build 17 Build Successful: |
Signed-off-by: TomFitzpatrick <[email protected]>
...sion-endpoint/src/main/kotlin/net/corda/libs/permissions/endpoints/v1/group/GroupEndpoint.kt
Outdated
Show resolved
Hide resolved
...sion-endpoint/src/main/kotlin/net/corda/libs/permissions/endpoints/v1/group/GroupEndpoint.kt
Show resolved
Hide resolved
.../main/kotlin/net/corda/libs/permissions/endpoints/v1/group/types/GroupContentResponseType.kt
Outdated
Show resolved
Hide resolved
.../main/kotlin/net/corda/libs/permissions/endpoints/v1/group/types/GroupContentResponseType.kt
Outdated
Show resolved
Hide resolved
...nager/src/main/kotlin/net/corda/libs/permissions/manager/response/GroupContentResponseDto.kt
Outdated
Show resolved
Hide resolved
...pl/src/test/kotlin/net/corda/libs/permissions/manager/impl/PermissionGroupManagerImplTest.kt
Outdated
Show resolved
Hide resolved
...-manager/src/main/kotlin/net/corda/libs/permissions/manager/request/DeleteGroupRequestDto.kt
Show resolved
Hide resolved
…line Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[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.
Looks fine, one suggestion posted.
...sion-endpoint/src/main/kotlin/net/corda/libs/permissions/endpoints/v1/group/GroupEndpoint.kt
Outdated
Show resolved
Hide resolved
… empty. Co-authored-by: Viktor Kolomeyko <[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
Signed-off-by: TomFitzpatrick <[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
Signed-off-by: TomFitzpatrick <[email protected]>
Signed-off-by: TomFitzpatrick <[email protected]>
Quality Gate passedIssues Measures |
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
This PR creates the GroupRestResource to allow creation and management of RBAC Groups. It includes all changes to support the Rest worker side of the work, up until sending the PermissionManagementRequest on Kafka. The DB side of this work will be continued in a following PR as this one is getting quite long.
This is supported by api changes in #6231.