diff --git a/src/internal/m365/collection/drive/collection.go b/src/internal/m365/collection/drive/collection.go index 423c43930..b29cb98be 100644 --- a/src/internal/m365/collection/drive/collection.go +++ b/src/internal/m365/collection/drive/collection.go @@ -265,7 +265,10 @@ func (oc *Collection) getDriveItemContent( // Skip big OneNote files as they can't be downloaded if clues.HasLabel(err, graph.LabelStatus(http.StatusServiceUnavailable)) && - oc.scope == CollectionScopePackage && *item.GetSize() >= MaxOneNoteFileSize { + // TODO: We've removed the file size check because it looks like we've seen persistent + // 503's with smaller OneNote files also. + // oc.scope == CollectionScopePackage && *item.GetSize() >= MaxOneNoteFileSize { + oc.scope == CollectionScopePackage { // FIXME: It is possible that in case of a OneNote file we // will end up just backing up the `onetoc2` file without // the one file which is the important part of the OneNote diff --git a/src/internal/m365/collection/drive/collection_test.go b/src/internal/m365/collection/drive/collection_test.go index 2e2f85160..9ced071f9 100644 --- a/src/internal/m365/collection/drive/collection_test.go +++ b/src/internal/m365/collection/drive/collection_test.go @@ -565,7 +565,7 @@ func (suite *GetDriveItemUnitTestSuite) TestGetDriveItem_error() { colScope: CollectionScopePackage, itemSize: 10, err: clues.New("small onenote error").Label(graph.LabelStatus(http.StatusServiceUnavailable)), - labels: []string{graph.LabelStatus(http.StatusServiceUnavailable)}, + labels: []string{graph.LabelStatus(http.StatusServiceUnavailable), graph.LabelsSkippable}, }, { name: "big OneNote file",