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

Add method to flush the list of logs #134

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MarceloRGonc
Copy link

@MarceloRGonc MarceloRGonc commented Feb 4, 2025

Description

This PR adds the feature to flush the messages list. This feature is relevant if logzio-js is being used in a serverless service, meaning there is no need to close the connection to ensure that messages are sent.

Also related to this issue: #101

What type of PR is this?

  • 🍕 Feature
  • 🐛 Bug Fix
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build / CI
  • ⏩ Revert

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help from somebody

@MarceloRGonc
Copy link
Author

@yotamloe Can you take a look at this? I'm open to discuss it.

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