Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

fix proxywasm leaks #224

Merged
merged 1 commit into from
May 9, 2024
Merged

fix proxywasm leaks #224

merged 1 commit into from
May 9, 2024

Conversation

bonifaido
Copy link
Member

@bonifaido bonifaido commented May 8, 2024 •
edited
Loading

Description

  • free proxywasm instances on exit
  • log compile errors, but don't fully compile proxywasm modules (pre-compile)
  • also implementing get nanoseconds in proxywasm

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@bonifaido bonifaido self-assigned this May 8, 2024
@bonifaido bonifaido marked this pull request as ready for review May 8, 2024 08:29
@bonifaido bonifaido requested a review from a team as a code owner May 8, 2024 08:29
@bonifaido bonifaido merged commit 90d7b5b into main May 9, 2024
5 checks passed
@bonifaido bonifaido deleted the proxywasm_free branch May 9, 2024 09:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@baluchicken baluchicken baluchicken approved these changes

Assignees

@bonifaido bonifaido

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bonifaido @baluchicken