Skip to content

fix(admin): validate execution id UUID format before making S3 request#619

Merged
Guimove merged 2 commits intomainfrom
fix/validate-exec-id-format-before-request
Mar 18, 2026
Merged

fix(admin): validate execution id UUID format before making S3 request#619
Guimove merged 2 commits intomainfrom
fix/validate-exec-id-format-before-request

Conversation

@Guimove
Copy link
Contributor

@Guimove Guimove commented Mar 18, 2026

Summary

  • Add UUID format validation on the execution id before sending the HTTP request
  • Returns a clear error message with a hint to remove the timestamp suffix for cluster execution ids
  • Remove the redundant info log that was shown after the 400 error (now surfaced upfront)

Test plan

  • Run qovery admin download-s3-archive -e <uuid>-<timestamp> → should fail immediately with a clear error, no HTTP request made
  • Run qovery admin download-s3-archive -e <valid-uuid> → should work as before

@Guimove Guimove merged commit fc89fdc into main Mar 18, 2026
5 checks passed
@Guimove Guimove deleted the fix/validate-exec-id-format-before-request branch March 18, 2026 11:39
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.

2 participants