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

Introduce HTTP logging event handler #5818

Closed
wants to merge 3 commits into from

Conversation

silencily
Copy link
Contributor

🔍 Description

Issue References 🔗

Describe Your Solution 🔧

Add event http logger handler

Types of changes 🔖

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Behavior Without This Pull Request ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests


Checklists

📝 Author Self Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • This patch was not authored or co-authored using Generative Tooling

📝 Committer Pre-Merge Checklist

  • Pull request title is okay.
  • No license issues.
  • Milestone correctly set?
  • Test coverage is ok
  • Assignees are selected.
  • Minimum number of approvals
  • No changes are requested

Be nice. Be informative.

@pan3793
Copy link
Member

pan3793 commented Dec 5, 2023

@silencily thanks for your contribution, can we have a doc for this feature? What should be HTTP server be like?

@bowenliang123 bowenliang123 marked this pull request as draft December 6, 2023 10:19
@bowenliang123 bowenliang123 changed the title Add event http logger handler Introduce ElasticSearch event logger Dec 6, 2023
@bowenliang123 bowenliang123 changed the title Introduce ElasticSearch event logger Introduce HTTP event logger Dec 6, 2023
@bowenliang123 bowenliang123 changed the title Introduce HTTP event logger Introduce HTTP logging event handler Dec 6, 2023
@silencily silencily closed this by deleting the head repository Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants