Skip to content

[doc] Updated streaming response limit to 200 MB #548

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 6, 2025

Conversation

dsplatonov
Copy link
Contributor

Motivation:

AWS Lambda response streaming now supports a default maximum response payload size of 200 MB (increased from 20 MB). This update aligns the documentation with the new AWS limits.

Modifications:

  • Updated readme.md: Changed "soft limit of 20 MB" to "default maximum response payload size of 200 MB"
  • Updated Examples/Streaming/README.md: Same change as above
  • Updated terminology to match AWS official documentation

Result:

Documentation now accurately reflects the current AWS Lambda streaming response limits, enabling users to understand they can stream up to 200 MB payloads.

Reference: https://aws.amazon.com/about-aws/whats-new/2025/07/aws-lambda-response-streaming-200-mb-payloads/

- Updated readme.md and Examples/Streaming/README.md
- Changed from 'soft limit of 20 MB' to 'default maximum response payload size of 200 MB'
- Aligns with AWS announcement about increased Lambda streaming limits"
@sebsto
Copy link
Contributor

sebsto commented Aug 5, 2025

Thank you @dsplatonov !
Your change will fix #546

But I think there is a bit more than the doc to check. I know the LambdaRuntimeClient struct has several references to the 6Mb payload limit for non-streaming operations. We need to make sure there is no 20Mb limit hardcoded here and there.

@sebsto sebsto added the semver/none No version bump required. label Aug 5, 2025
@sebsto sebsto self-assigned this Aug 5, 2025
@sebsto sebsto added this to the 2.0 milestone Aug 5, 2025
@dsplatonov
Copy link
Contributor Author

Yea, I’ve checked all files, found 20 mb only in documentation, 6mb limit is not changed, so I didn’t change it in code:)

@sebsto sebsto self-requested a review August 6, 2025 07:57
Copy link
Contributor

@sebsto sebsto left a comment

Choose a reason for hiding this comment

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

LGTM. I also checked for hard-code 20Mb limit in the code and could not find any.

@sebsto sebsto merged commit af7e9db into swift-server:main Aug 6, 2025
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants