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

Omit bogus items in OneDrive list services #16952

Open
AliveDevil opened this issue Mar 14, 2025 · 0 comments
Open

Omit bogus items in OneDrive list services #16952

AliveDevil opened this issue Mar 14, 2025 · 0 comments
Assignees
@AliveDevil
Labels
onedrive OneDrive Protocol Implementation
Milestone
9.1.4

Comments

@AliveDevil
Copy link
Contributor

AliveDevil commented Mar 14, 2025 •
edited
Loading 

2025-03-13 14:06:15,954 [Thread-16] DEBUG ch.iterate.mountainduck.fs.FilesystemIncrementalListProgressListener - Return next Path{path='/Shared/***', type=[directory, shared]}
2025-03-13 14:06:15,954 [Thread-16] DEBUG Ch.Iterate.Mountainduck.Fs.Cbfs.CBFSFilesystem - ToAttributes: PathAttributes{***, versionId='null', fileId='***/null', ***}, ***

Proposal to add some kind of fast-rejection/validation in GraphSession to straight up reject DriveItem.Metadata, when they are provably garbage (including logging the bad entry).

I.e. Add hook to AbstractListService.java#L58 to quickly verify that an item may be accessible.
@AliveDevil AliveDevil added the onedrive OneDrive Protocol Implementation label Mar 14, 2025
@AliveDevil AliveDevil added this to the 9.1.4 milestone Mar 14, 2025
@AliveDevil AliveDevil self-assigned this Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AliveDevil AliveDevil

Labels
onedrive OneDrive Protocol Implementation
Projects
None yet
Milestone
9.1.4
Development

No branches or pull requests
1 participant
@AliveDevil