-
Notifications
You must be signed in to change notification settings - Fork 13
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
[tests-only][full-ci]added bug demo test for removing systemtags app from whitelist for guest users #559
[tests-only][full-ci]added bug demo test for removing systemtags app from whitelist for guest users #559
Conversation
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 👍
5c22b84
to
a60deb8
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.
Other than small comments, LGTM 👍
a60deb8
to
a19749e
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
90cfaba
to
4375863
Compare
4375863
to
0e53490
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. During the development of the fix, the developer and QA etc can sort out if these system-tag requests by the guest should return 403 (forbidden) or 404 (not found), and adjust the scenario steps as needed.
Kudos, SonarCloud Quality Gate passed! |
Description
This PR adds bug demonstration test for removing
systemtag
app from whilelist for guest users, so that guest user cannot add, remove or view tags of a resource.Related Issue
Related to issue: #551
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: