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.

Sign up for GitHub

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

Jump to bottom

Authorization in rest layer automated test for audit logging #3099

Closed
3 tasks done
Tracked by #2590
peternied opened this issue Aug 4, 2023 · 0 comments · Fixed by #3192
Closed
3 tasks done
Tracked by #2590

Authorization in rest layer automated test for audit logging #3099

peternied opened this issue Aug 4, 2023 · 0 comments · Fixed by #3192
Assignees
@DarshitChanpura
Labels
triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.

Comments

@peternied
Copy link
Member

peternied commented Aug 4, 2023 •
edited
Loading

We've got unit tests that verify that the audit log sink is firing and we should be sure that the audit log entries are generated when using authZ in the rest layer.

Exit Criteria

  • Integration test that verifies authorization in the rest layer success creates an audit log message
  • Integration test that verifies authorization in the rest layer failure creates an audit log message
  • Integration test that verifies authorization in rest layer vs transport layer audit message bodies are identical
@peternied peternied mentioned this issue Aug 4, 2023
Closed
15 tasks
@github-actions github-actions bot added the untriaged Require the attention of the repository maintainers and may need to be prioritized label Aug 4, 2023
@peternied peternied changed the title Add end to end test case that verifies audit logging [RestAuthz] Add end to end test case that verifies audit logging Aug 4, 2023
@peternied peternied changed the title [RestAuthz] Add end to end test case that verifies audit logging Authorization in rest layer automated test for audit logging Aug 4, 2023
This was referenced Aug 4, 2023
Closed
Closed
Merged
@cwperks cwperks added triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable. and removed untriaged Require the attention of the repository maintainers and may need to be prioritized labels Aug 7, 2023
@DarshitChanpura DarshitChanpura mentioned this issue Aug 16, 2023
Merged
3 tasks
DarshitChanpura added a commit that referenced this issue Aug 21, 2023
@DarshitChanpura
Adds a test to check similarity between audit logs generated from RES…
b0e3cdc 
…T layer and TRANSPORT layer (#3192)

- Resolves #3099

Signed-off-by: Darshit Chanpura <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DarshitChanpura DarshitChanpura

Labels
triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.
Projects
None yet
Milestone
No milestone
3 participants
@peternied @cwperks @DarshitChanpura