Skip to content

feat: extend request logs with auth & DB info #17304

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
Merged
merged 13 commits into from
Apr 15, 2025
Merged

feat: extend request logs with auth & DB info #17304

merged 13 commits into from
Apr 15, 2025
+336 −35

Conversation

ibetitsmike
Copy link
Contributor

@ibetitsmike ibetitsmike commented Apr 9, 2025 •
edited
Loading

Closes #16903

@ibetitsmike ibetitsmike changed the title expanded user query with email on authorization and extracted actor i… feat: request logs are extended with user details Apr 9, 2025
@ibetitsmike ibetitsmike force-pushed the mike/16903-extend-request-logs branch from 1263c3c to 9bbef21 Compare April 10, 2025 06:08
@ibetitsmike ibetitsmike changed the title feat: request logs are extended with user details feat: request logs are extended with authentication info Apr 10, 2025
@ibetitsmike ibetitsmike changed the title feat: request logs are extended with authentication info feat: request logs are extended with auth & DB info Apr 10, 2025
deansheather
deansheather reviewed Apr 10, 2025
View reviewed changes
@ibetitsmike ibetitsmike force-pushed the mike/16903-extend-request-logs branch 2 times, most recently from e42dc76 to 942437f Compare April 10, 2025 14:30
@ibetitsmike ibetitsmike marked this pull request as ready for review April 11, 2025 10:47
@ibetitsmike ibetitsmike changed the title feat: request logs are extended with auth & DB info feat: extend request logs with auth & DB info Apr 11, 2025
@ibetitsmike ibetitsmike force-pushed the mike/16903-extend-request-logs branch from 4390c4d to 8ef5646 Compare April 14, 2025 10:31
deansheather
deansheather approved these changes Apr 15, 2025
View reviewed changes
Copy link
Member

@deansheather deansheather left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@deansheather deansheather requested a review from Emyrk April 15, 2025 01:57
deansheather
deansheather approved these changes Apr 15, 2025
View reviewed changes
@ibetitsmike ibetitsmike merged commit 06d3915 into main Apr 15, 2025
36 of 37 checks passed
@ibetitsmike ibetitsmike deleted the mike/16903-extend-request-logs branch April 15, 2025 11:27
@github-actions github-actions bot locked and limited conversation to collaborators Apr 15, 2025
@ibetitsmike
Copy link
Contributor Author

/cherry-pick release/2.19

@ibetitsmike
Copy link
Contributor Author

/cherry-pick release/2.21

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@deansheather deansheather deansheather approved these changes

@Emyrk Emyrk Awaiting requested review from Emyrk

Assignees

@ibetitsmike ibetitsmike

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Request logs should contain some resolved information about DB objects
2 participants
@ibetitsmike @deansheather