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.

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

Feature/update server app logging #395

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

courtneymyers
Copy link
Collaborator

Related Issues:

  • CSBAPP-276

Main Changes:

Update server app's BAP requests to pass in request object in all logged messages.

NOTE: we were already doing this in most requests in the server app...this just updates any lingering requests that weren't already passing in the request object, so more data is available in the server logs.

Steps To Test:

  1. Navigate the app, and ensure everything still functions as expected.
  2. Look at the server logs in Kibana and ensure the BAP requests now all contain the request object.

@courtneymyers courtneymyers merged commit ed093ef into develop Feb 28, 2024
1 check passed
@courtneymyers courtneymyers deleted the feature/update-server-app-logging branch February 28, 2024 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant