Skip to content

Match real Vuforia's reco counts report URL, expiry and not-ready response - #3570

Merged
adamtheturtle merged 4 commits into
mainfrom
issue-3364
Sep 8, 2026
Merged

Match real Vuforia's reco counts report URL, expiry and not-ready response#3570
adamtheturtle merged 4 commits into
mainfrom
issue-3364

Conversation

@adamtheturtle

Copy link
Copy Markdown
Member

Closes #3364.

Probes against a real database settled all three open items: a not-ready report is a 404 with the S3 NoSuchKey XML document, storage checks X-Amz-Date plus X-Amz-Expires before the signature, and a second request in the same hour serves the file which the first request generated. The mock now names report files YYYY-mm-dd-HH.csv for the current month and YYYY-mm.csv for the previous month under /reports/{database_id}/, returns a URL with the seven real X-Amz-* query parameters, honors the expiry with the S3 AccessDenied documents, and gives the NoSuchKey document until the report is generated. The download tests now also run against real Vuforia, with new coverage of the URL shape, file names, same-hour collisions, expiry and the missing query, and mock-only tests create their own database via a new add_cloud_database helper because the shared fixture database reuses one ID across a run. The differences doc describes the real URL shape and the deliberate signature difference, and the unverified-behavior entry for the not-ready response is removed.

🤖 Generated with Claude Code

…ponse

Closes #3364.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
adamtheturtle and others added 3 commits September 8, 2026 09:56
# Conflicts:
#	docs/source/unverified-behavior.rst
…rget manager before Flask usage tests

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
# Conflicts:
#	tests/mock_vws/test_reco_counts_report.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reco counts presigned URL name, expiry and not-ready behaviour differ from real Vuforia

1 participant