QM: Directory traversals prematurely finish if a directory entry is modified and then deleted
Categories
(Core :: Storage: Quota Manager, defect, P3)
Tracking
()
People
(Reporter: janv, Unassigned)
Details
(Keywords: leave-open, Whiteboard: [storage-jv])
Attachments
(2 files)
I've noticed this when I was working on fixes for ignoring unknown files.
Basically, if an origin directory is missing the metadata file during a clear operation, the metadata will be restored (which probably modifies the current directory entry of the traversal). If the origin directory also matches the scope, it will be deleted. However, the directory traversal finishes after that.
I verified this on the try server and it's a problem on all platforms:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f71c07f2b650551d3cacd553931ff73b56b5bb4d
For now, I'll just modify a test with a TODO comment, so we don't forget about it.
Reporter | ||
Comment 1•4 years ago
|
||
Updated•4 years ago
|
Reporter | ||
Comment 2•4 years ago
|
||
Depends on D75786
Reporter | ||
Updated•4 years ago
|
Pushed by jvarga@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/528ca16f21cb Clean up the test for clearStoragesForOriginAttributesPattern(); r=dom-workers-and-storage-reviewers,ttung https://hg.mozilla.org/integration/autoland/rev/bc0a37f04241 Rename clearStorageForPrincipal to clearStoragesForPrincipal in tests; r=dom-workers-and-storage-reviewers,ttung
Comment 4•4 years ago
|
||
bugherder |
Comment 5•3 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:janv, maybe it's time to close this bug?
Comment 6•3 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:janv, maybe it's time to close this bug?
Comment 7•2 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:janv, maybe it's time to close this bug?
Comment 8•2 years ago
|
||
As the patches seem to touch only tests, I assume this is still open. Jan, are you planning to work on this?
Updated•2 years ago
|
Reporter | ||
Updated•2 years ago
|
Comment 9•1 year ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:edenchuang, maybe it's time to close this bug?
For more information, please visit auto_nag documentation.
Comment 10•1 year ago
|
||
(In reply to Release mgmt bot [:suhaib / :marco/ :calixte] from comment #9)
:edenchuang, maybe it's time to close this bug?
No.
Description
•