Skip to content

Fix Bug #3355: Fix that long running big upload (220GB+) fails because of an expired access token#3361

Merged
abraunegg merged 19 commits intomasterfrom
fix-issue-3355
Jul 3, 2025
Merged

Fix Bug #3355: Fix that long running big upload (220GB+) fails because of an expired access token#3361
abraunegg merged 19 commits intomasterfrom
fix-issue-3355

Conversation

@abraunegg
Copy link
Owner

  • Revert back to v2.5.5 performSessionFileUpload() and apply minimal change for upload session offset handling to prevent desynchronisation on large files

* Revert back to v2.5.5 performSessionFileUpload() and apply minimal change for upload session offset handling to prevent desynchronisation on large files
@abraunegg abraunegg added this to the v2.5.7 milestone Jun 28, 2025
@abraunegg abraunegg linked an issue Jun 28, 2025 that may be closed by this pull request
@abraunegg abraunegg changed the title Fix Bug #3355: Update performSessionFileUpload() handling Fix Bug #3355: Fix that long running big upload (250GB+) fails because of an expired access token Jun 29, 2025
* Add specific 403 handler for when the upload session URL itself expires
@github-actions

This comment has been minimized.

* Cleanup 403 messaging
* Add 'file_fragment_size'
@github-actions

This comment has been minimized.

abraunegg added 16 commits July 1, 2025 17:28
* Clean up debug logging output
* Add 'tempauth' to spelling words
* Update documentation URL's
* Ensure that on each fragment upload, whilst the application is using the 'tempauth' for session upload, the global OAuth2 token needs to be checked for validity and refreshed if required
* Add limit check for 'file_fragment_size' option
* Add to default 'config' file
* Update documentation for 'file_fragment_size'
* Add 'file_fragment_size' to --display-config output
* Add --file-fragment-size option to enable use via Docker option
* Add to manpage
* Update Docker entrypoint
* Update Docker | Podman documentation
* Update logging output to include connection method to URL
* Update Upload Session URL expiry update to include UTC and LocalTime values
* Update comment which was dropped / missed
* Clarify that this is the OAuth2 Access Token
* Clarify that the expiry timestamp is localTime
* Update PR with dynamic use of fragment size if fileSize > 1GB
* Increase upperlimit to 59
* Enforce multiple 320KiB for fragment size
* Fix Docker entrypoint and confirm working for ONEDRIVE_FILE_FRAGMENT_SIZE
* Change 'defaultMaxFileFragmentSize' to 60
* Revise fragmentSize calculation to be as close to 60 MiB as possible without breaching Microsoft documented threshold
* Change threshold for using larger fragment size to files > 100MiB
@abraunegg abraunegg merged commit ea7c3ab into master Jul 3, 2025
10 checks passed
@abraunegg abraunegg deleted the fix-issue-3355 branch July 4, 2025 10:47
@abraunegg abraunegg changed the title Fix Bug #3355: Fix that long running big upload (250GB+) fails because of an expired access token Fix Bug #3355: Fix that long running big upload (220GB+) fails because of an expired access token Jul 7, 2025
@abraunegg
Copy link
Owner Author

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

Repository owner locked and limited conversation to collaborators Jul 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Big upload fails because of an expired access token

1 participant