feat: add PresignInlineURL method for inline content display #97

Merged
argoyle merged 1 commits from feat-presign-inline into main 2026-04-16 08:52:40 +00:00
Owner

Summary

  • Add PresignInlineURL(ctx, key, contentType) method that generates presigned URLs with Content-Disposition: inline and optional Content-Type override
  • Browsers will render content (e.g. PDFs) directly in iframes instead of triggering download dialogs
  • Existing PresignURL remains unchanged

Context

The document-service uses presigned S3 URLs to display PDFs in iframes. Without Content-Disposition: inline, the browser triggers a download dialog instead of rendering the PDF.

🤖 Generated with Claude Code

## Summary - Add `PresignInlineURL(ctx, key, contentType)` method that generates presigned URLs with `Content-Disposition: inline` and optional `Content-Type` override - Browsers will render content (e.g. PDFs) directly in iframes instead of triggering download dialogs - Existing `PresignURL` remains unchanged ## Context The document-service uses presigned S3 URLs to display PDFs in iframes. Without `Content-Disposition: inline`, the browser triggers a download dialog instead of rendering the PDF. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
argoyle added 1 commit 2026-04-16 08:45:51 +00:00
feat: add PresignInlineURL method for inline content display
storage / test (pull_request) Successful in 1m41s
storage / vulnerabilities (pull_request) Successful in 1m46s
pre-commit / pre-commit (pull_request) Successful in 6m10s
4199fc84e7
Generates presigned URLs with Content-Disposition: inline and optional
Content-Type override, so browsers render content (e.g. PDFs) directly
instead of triggering a download dialog.
argoyle scheduled this pull request to auto merge when all checks succeed 2026-04-16 08:46:15 +00:00
argoyle merged commit 862ec3f7bc into main 2026-04-16 08:52:40 +00:00
argoyle deleted branch feat-presign-inline 2026-04-16 08:52:41 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: unboundsoftware/storage#97