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

Fix memory leak in Micro #16404

Merged
merged 6 commits into from
Aug 15, 2023
Merged

Fix memory leak in Micro #16404

merged 6 commits into from
Aug 15, 2023

Conversation

Jeckerson
Copy link
Member

@Jeckerson Jeckerson commented Aug 15, 2023 •
edited by niden
Loading

Hello!

  • Type: bug fix
  • Link to issue:

In raising this pull request, I confirm the following:

  • I have read and understood the Contributing Guidelines
  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I have updated the relevant CHANGELOG
  • I have created a PR for the documentation about this change

Thanks

@Jeckerson
Remove set of 'application' service
05589ec 
This causes memory leak
@Jeckerson Jeckerson added bug A bug report 5.0 The issues we want to solve in the 5.0 release labels Aug 15, 2023
@Jeckerson Jeckerson requested a review from niden August 15, 2023 15:59
@Jeckerson Jeckerson self-assigned this Aug 15, 2023
@Jeckerson Jeckerson changed the title Remove set of 'application' service Fix memory leak in Micro Aug 15, 2023
@Jeckerson Jeckerson merged commit f6f342e into 5.0.x Aug 15, 2023
2 checks passed
@Jeckerson Jeckerson deleted the fix-memory-leak branch August 15, 2023 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@niden niden niden approved these changes

Assignees

@Jeckerson Jeckerson

Labels
5.0 The issues we want to solve in the 5.0 release bug A bug report
Projects
Phalcon v5
Status: Released
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Jeckerson @niden