Skip to content

addons(cellar): Add section about Pre-signed URL and checksum validation - #971

Merged
Kirbeerus merged 1 commit into
mainfrom
Cellar-pre-signed-url-upload-checksum
Aug 18, 2026
Merged

Kirbeerus merged 1 commit into
mainfrom
Cellar-pre-signed-url-upload-checksum

Conversation

@Kirbeerus

@Kirbeerus Kirbeerus commented Jun 18, 2026 •

Copy link
Copy Markdown
Contributor

📝 What does this PR do?

Add section in the cellar documentation about presigned URL and that you can validate with checksum verification.
Add precision that it work fully with MD5. But using SHA256 only verify the header and not the file content

🔗 Related Issue (if applicable)

  • Closes #
  • Related to #

🧪 Type of Change

  • ⚠️ Bug fix
  • 📅 Changelog update
  • 📚 Documentation update
  • ✨ New content/feature
  • 🔧 Technical/maintenance

✅ Quick Checklist


👥 Reviewers

@CleverCloud/reviewers


📋 For major changes (click to expand)

Additional testing performed

Describe any specific testing done for complex changes

Screenshots

Add screenshots for visual/layout changes

Breaking changes

List any breaking changes or migration notes

@github-actions

github-actions Bot commented Jun 18, 2026 •

Copy link
Copy Markdown

👋` Review app deleted

You closed this PR and deleted the review app.

@Kirbeerus
Kirbeerus force-pushed the Cellar-pre-signed-url-upload-checksum branch from 046699b to ae427f3 Compare June 18, 2026 10:50
@Kirbeerus
Kirbeerus requested a review from a team June 18, 2026 10:54
@Kirbeerus Kirbeerus self-assigned this Jun 18, 2026
@cnivolle

cnivolle commented Aug 3, 2026

Copy link
Copy Markdown
Member

All good. On a side note, in the code section: var d'envs could be used.
wdyt ?

@Kirbeerus
Kirbeerus merged commit 9de9f0f into main Aug 18, 2026
5 checks passed
@Kirbeerus
Kirbeerus deleted the Cellar-pre-signed-url-upload-checksum branch August 18, 2026 13:11

This branch was previously deployed

1 inactive deployment
update doc — ae427f3a Deployed Jun 18, 2026 by Kirbeerus via Test changed-files #1482
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