Switch use of IndexedDB openCursor to openKeyCursor for fetching keys #874
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
First off, first time contributing to this project. Let me know if I'm doing anything wrong.
A little bit of background:
I'm running into a problem with some of my production IndexedDB databases which seems to be related to accessing the whole db with
openCursor
. I haven't been able to figure out why I'm having this problem or how to reproduce it with a db that doesn't contain sensitive information, but I did notice that usingopenKeyCursor
for thekey
andkeys
calls (in the IndexedDB driver) lets me work around the problem for now.Is there a particular reason we're using
openCursor
instead ofopenKeyCursor
for thekey
andkeys
call in the IndexedDB driver? Is there compatibility issues I'm not aware of? As far as I know, usingopenKeyCursor
in this case would be much faster and is just more appropriate since we don't need any actual values for these calls.