Skip to content

External storage and codec server sample#307

Open
jmaeagle99 wants to merge 1 commit into
temporalio:mainfrom
jmaeagle99:extstore
Open

External storage and codec server sample#307
jmaeagle99 wants to merge 1 commit into
temporalio:mainfrom
jmaeagle99:extstore

Conversation

@jmaeagle99
Copy link
Copy Markdown
Contributor

What was changed

Add sample for external storage with codec server that supports multiple namespaces and downloads.

Why?

Provide a baseline reference for how to use external storage with the S3 driver and an updated codec server.

Checklist

  1. How was this tested: Locally ran; investigating possibility of adding tests
  2. Any docs updates needed? No

Comment thread external_storage/handler.py
@semgrep-managed-scans
Copy link
Copy Markdown

Semgrep found 6 ssc-66c432ac-3608-4a86-bf47-45df36e8c883 findings:

Risk: Affected versions of aiohttp are vulnerable to Allocation of Resources Without Limits or Throttling / Improper Handling of Highly Compressed Data (Data Amplification). Unbounded decompression of compressed request bodies in aiohttp's server allows an attacker to send a specially crafted "zip bomb" (e.g. via Content-Encoding: gzip) that expands to an arbitrarily large payload in memory, exhausting host resources and causing a denial-of-service.

Fix: Upgrade this library to at least version 3.13.3 at samples-python/uv.lock:65.

Reference(s): GHSA-6mq8-rvhq-8wgg, CVE-2025-69223

@jmaeagle99 jmaeagle99 marked this pull request as ready for review May 14, 2026 17:02
@jmaeagle99 jmaeagle99 requested a review from a team as a code owner May 14, 2026 17:02
Copy link
Copy Markdown

@cconstable cconstable left a comment

Choose a reason for hiding this comment

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

👍 Very neat. Learned about X-Namespace and preserveStorageRefs from reviewing this.

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