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

Logging framework - Add a endrequest log line which should be a common line for accumulating all parameters #3419

Closed
lsampras opened this issue Jan 22, 2024 · 0 comments · Fixed by #3437
Closed

Logging framework - Add a endrequest log line which should be a common line for accumulating all parameters #3419

lsampras opened this issue Jan 22, 2024 · 0 comments · Fixed by #3437
Assignees
@lsampras
Labels
A-framework Area: Framework

Comments

@lsampras
Copy link
Member

lsampras commented Jan 22, 2024 •
edited
Loading

Currently we use the EndRequest filter by actix to rely on this, but we need a log line that represents a request uniquely & contains all the key metadata about the request

the list of fields that we need

  • Flow (already present)
  • payment_id
  • connector_name
  • payment_method
@lsampras lsampras self-assigned this Jan 22, 2024
@lsampras lsampras changed the title Logging framework - Add golden log line which should be a common line for accumulating all parameters Logging framework - Add a endrequest log line which should be a common line for accumulating all parameters Jan 22, 2024
@lsampras lsampras added the A-framework Area: Framework label Jan 22, 2024
@lsampras lsampras linked a pull request Jan 24, 2024 that will close this issue
feat(logging): add a logging middleware to log all api requests #3437
Merged
15 tasks
@lsampras lsampras linked a pull request Jan 29, 2024 that will close this issue
fix(logging): add flow to persistent logs fields #3472
Merged
15 tasks
@lsampras lsampras removed a link to a pull request Jan 30, 2024
fix(logging): add flow to persistent logs fields #3472
Merged
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lsampras lsampras

Labels
A-framework Area: Framework
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(logging): add a logging middleware to log all api requests juspay/hyperswitch
1 participant
@lsampras