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

Added RequestId to cumulus format log messages #275

Closed
bbuechler opened this issue Oct 28, 2020 · 1 comment
Closed

Added RequestId to cumulus format log messages #275

bbuechler opened this issue Oct 28, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@bbuechler
Copy link
Contributor

seems obvious in retrospect, but would be useful for capturing the final result.

Alternative solution:

  1. detect a dict object being passed into a call logger call...
  2. add each element of the dict to the log JSON...
  3. have {"message": "<VALUE>"} be something to the affect of "cumulus log response payload"

def cumulus_log_message(outcome: str, code: int, http_method:str, k_v: dict):

@bbuechler bbuechler added the enhancement New feature or request label Oct 28, 2020
@benbart benbart self-assigned this Apr 28, 2021
bbuechler added a commit that referenced this issue May 4, 2021
Added RequestID to cumulus log message #275
@bbuechler
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants