Skip to content
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

Add response streaming support #1635

Open
2 tasks
plaisted opened this issue Dec 7, 2023 · 5 comments
Open
2 tasks

Add response streaming support #1635

plaisted opened this issue Dec 7, 2023 · 5 comments
Labels
feature-request A feature should be added or improved. module/lambda-client-lib p2 This is a standard priority issue queued

Comments

@plaisted
Copy link

plaisted commented Dec 7, 2023

Describe the feature

Add support for using lambda response streaming when returning streams in a dotnet lambda function.

https://aws.amazon.com/blogs/compute/introducing-aws-lambda-response-streaming/

Use Case

I would like to stream large S3 objects to http clients using a lambda URL.

Proposed Solution

Add functionality to enable returned streams to be sent using application/vnd.awslambda.http-integration-response content type and prefixed with required JSON prelude / null bytes. Allow status code / http headers to be configured and added to JSON prelude.

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

AWS .NET SDK and/or Package version used

Amazon.Lambda.RuntimeSupport 1.10.0

Targeted .NET Platform

.net 8

Operating System and version

debian container in lambda

@plaisted plaisted added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Dec 7, 2023
@ashishdhingra ashishdhingra added module/lambda-client-lib needs-review and removed needs-triage This issue or PR still needs to be triaged. labels Dec 7, 2023
@ashishdhingra
Copy link
Contributor

Discussion #1632 opened few days ago.

@plaisted
Copy link
Author

plaisted commented Dec 8, 2023

I was able to get this working with pretty minimal changes. I'll push it somewhere in case anyone wants to replicate until it's officially supported.

@plaisted
Copy link
Author

plaisted commented Dec 8, 2023

Certainly not hardened but is working for my use case of returning dynamic http content from a lambda URL:
plaisted@3d45f5a

Usable by returning new StreamedResponse class which is a basic wrapper of a normal .net stream + http info:

    [LambdaFunction]
    public async Task<StreamedResponse> CustomerCDN(APIGatewayHttpApiV2ProxyRequest req, ILambdaContext ctx)
    {
        // dummy content
        await Task.Yield();
        var content = new MemoryStream(Encoding.UTF8.GetBytes("Example content!!".ToString()));

        // return using lambda response streaming
        return new StreamedResponse(content)
        {
            Headers = new Dictionary<string, string>
            {
                ["Content-Type"] = "text/plain; charset=utf-8",
                ["Cache-Control"] = "no-store, private, stale-if-error=0"
            }
        };
    }

@ashishdhingra ashishdhingra added p2 This is a standard priority issue queued and removed needs-review labels Dec 8, 2023
@paolofulgoni
Copy link

Another use case would be streaming an LLM response (e.g. with Anthropic Claude)

@normj
Copy link
Member

normj commented Apr 1, 2024

@paolofulgoni Good use case. Definitely feature I want to get to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A feature should be added or improved. module/lambda-client-lib p2 This is a standard priority issue queued
Projects
None yet
Development

No branches or pull requests

4 participants