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

Memory leak #354

Closed
nuevoalex opened this issue Apr 14, 2020 · 2 comments
Closed

Memory leak #354

nuevoalex opened this issue Apr 14, 2020 · 2 comments
Labels
Bug

Comments

@nuevoalex
Copy link

In IDBDatabase.js an array is kept which stores all transactions:

https://github.com/axemclion/IndexedDBShim/blob/7a2bd336cc0f60245ec308b25b112d857315c90a/src/IDBDatabase.js#L36

This array never has elements removed and will grow indefinitely as an application runs resulting in out of control memory growth.
@brettz9 brettz9 added the Bug label Apr 14, 2020
@brettz9
Copy link
Collaborator

brettz9 commented Apr 14, 2020

Thank you for the report. Please note that while I am maintaining the issues here for now, all further work I plan to do (including accepting of PRs) is on https://github.com/indexeddbshim/indexeddbshim .

@brettz9 brettz9 closed this as completed in c7cb378 Aug 2, 2024
brettz9 added a commit that referenced this issue Aug 2, 2024
@brettz9
fix: remove transaction from __transactions tracking array after tr…
5b7d92d 
…ansaction completion; fixes #354
brettz9 added a commit that referenced this issue Aug 2, 2024
@brettz9
fix: remove transaction from __transactions tracking array after tr…
7c50f58 
…ansaction completion; fixes #354
@brettz9
Copy link
Collaborator

brettz9 commented Aug 2, 2024

Released as part of 15.0.1 . Thanks for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@brettz9 @nuevoalex