Skip to content

Conversation

@xGeorge
Copy link

@xGeorge xGeorge commented Oct 20, 2025

Summary

Hello—iOS engineer here. Let me handle nit comments in a follow-up PR.
While profiling the app, I saw multiple NSOperation-related memory leaks in this library

@snyk-sa-branch
Copy link

snyk-sa-branch commented Oct 20, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Licenses 0 0 0 0 0 issues
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@xGeorge
Copy link
Author

xGeorge commented Oct 20, 2025

@branch-sdkteam

@xGeorge
Copy link
Author

xGeorge commented Oct 27, 2025

@branch-sdkteam
@branchlet

@xGeorge
Copy link
Author

xGeorge commented Nov 19, 2025

}];

// Optional: lower priority; it’s background I/O
op.queuePriority = NSOperationQueuePriorityLow;
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can remove this

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.

2 participants