apache / opendal

Apache OpenDAL: access data freely.
https://opendal.apache.org
Apache License 2.0
2.99k stars 417 forks source link

fix(core/services/onedrive): remove @odata.count for onedrive list op #4803

Closed imWildCat closed 1 week ago

imWildCat commented 1 week ago

This @odata.count does not seem to exist before in their official docs...

Recently I found they don't return this field anymore so all list op would raise errors.

I'm going to remove it.

In this repo, this field does not exist for all the time: https://github.com/sreeise/graph-rs-sdk/blob/afda1dcfbf5d491e56c6753152a1ce7e1c97fb44/test_files/drive_ep/drive_root_child.json#L4

imWildCat commented 1 week ago

for the record, I verified this change locally on my branch

image

end to end

I don't think a local run of the behavior test is needed because this PR is deleting an unused field

imWildCat commented 1 week ago

Thanks!

thanks for the quick review! 🫡