Skip to content

Conversation

@saw-jan
Copy link
Contributor

@saw-jan saw-jan commented Dec 9, 2025

Description

Changes:

  • retry on offset mismtach (409) while sending the last chunk twice
  • add test to check mismatch offset error

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation added

And user "Alice" sends a chunk to the last created TUS Location with offset "3" and data "4567890" using the WebDAV API
And the user waits for "2" seconds for postprocessing to finish
And user "Alice" sends a chunk to the last created TUS Location with offset "3" and data "0000000" using the WebDAV API
When user "Alice" sends a chunk to the last created TUS Location with offset "3" and data "0000000" with retry on offset mismatch using the WebDAV API
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this waits nicely until timeout instead of an explicit 2-second wait.

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
@saw-jan saw-jan force-pushed the test/add-mismatch-offset-test branch from 2dd3882 to ee16c05 Compare December 9, 2025 07:06
Copy link

@anon-pradip anon-pradip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔆 🚀

@saw-jan saw-jan merged commit a3a1397 into main Dec 9, 2025
59 checks passed
@saw-jan saw-jan deleted the test/add-mismatch-offset-test branch December 9, 2025 11:31
@openclouders openclouders mentioned this pull request Dec 9, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[QA] Fail api test while checking status code

3 participants