-
Notifications
You must be signed in to change notification settings - Fork 2
Lcra/change security visibility per role prod 14083 #836
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
Lcra/change security visibility per role prod 14083 #836
Conversation
modified tests in all services after role_viewer got the right to read security
added check in each services for each returned objects containing security if security was returned and the user has the role viewer then only minimal informations on security are returned default security is unchanged, the access control list contains only the user with the role viewer
f652815
to
4a44eb5
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.
LGTM
} | ||
|
||
@Test | ||
fun `test update solution with runTemplates specified`() { |
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.
failing test
} | ||
|
||
@Test | ||
fun `test update solution with empty runTemplates specified`() { |
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.
failing test
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.
after applying PROD-13734 fix and correcting versions seems to me ok for merging
No description provided.