Skip to content

[FIX] RocketChat.settings.get causing memory leak (sometimes)#11487

Merged
ggazzo merged 9 commits intodevelopfrom
setting-leak
Jul 19, 2018
Merged

[FIX] RocketChat.settings.get causing memory leak (sometimes)#11487
ggazzo merged 9 commits intodevelopfrom
setting-leak

Conversation

@ggazzo
Copy link
Copy Markdown
Member

@ggazzo ggazzo commented Jul 17, 2018

No description provided.

@ggazzo ggazzo requested a review from sampaiodiego July 17, 2018 18:01
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11487 July 17, 2018 18:01 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-11487 July 17, 2018 18:06 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-11487 July 17, 2018 18:13 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-11487 July 17, 2018 21:40 Inactive
@RocketChat RocketChat deleted a comment Jul 17, 2018
@RocketChat RocketChat deleted a comment Jul 17, 2018
@theorenck theorenck modified the milestones: 0.67.0, 0.68.0 Jul 19, 2018
@Hudell
Copy link
Copy Markdown
Contributor

Hudell commented Jul 19, 2018

I noticed that it only checks if the username is exactly the same as the one on the blacklist, but this was already the behaviour of this feature before this change.

@ggazzo ggazzo merged commit 146837b into develop Jul 19, 2018
@ggazzo ggazzo deleted the setting-leak branch July 19, 2018 16:57
@sampaiodiego sampaiodiego mentioned this pull request Jul 28, 2018
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.

4 participants