Skip to content

feat: Http Api lambda handler#86

Merged
floydspace merged 8 commits intomainfrom
feat/83-http-api-lambda
Apr 28, 2025
Merged

feat: Http Api lambda handler#86
floydspace merged 8 commits intomainfrom
feat/83-http-api-lambda

Conversation

@floydspace
Copy link
Owner

issue #83

@changeset-bot
Copy link

changeset-bot bot commented Dec 1, 2024

🦋 Changeset detected

Latest commit: 845ac99

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@effect-aws/lambda Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@floydspace floydspace mentioned this pull request Dec 1, 2024
@arjunyel
Copy link

Thank you for the excellent work! Is this blocked on anything?

Looking through the code, one thing that might be missing are Lambda Function URL events

import type { LambdaFunctionURLEvent } from "aws-lambda";

https://docs.aws.amazon.com/lambda/latest/dg/urls-configuration.html

@floydspace
Copy link
Owner Author

it is a bit blocking to make a decision if we need to handle all the events (I feel like supporting events like sns/sqs and etc. are not really usefull)

  • lack of proper testing.

I will do my best to prioritoze it.
And thanks for your interest, it's good to see it is awaited

@arjunyel
Copy link

it is a bit blocking to make a decision if we need to handle all the events (I feel like supporting events like sns/sqs and etc. are not really usefull)

  • lack of proper testing.

I will do my best to prioritoze it. And thanks for your interest, it's good to see it is awaited

Thank you so much friend! I can't speak for others but for my HttpApis I have been using Lamda function URLs. I don't think people would use an http API with a sns and sqs events. I think if you support function URLs and API gateway that would cover most use cases.

One thing that may make things more complicated is supporting Lambda streaming. Here is how Hono handles it: https://github.com/honojs/hono/blob/15a83b125cae5a772f265ce448974c03efa5e3bc/src/adapter/aws-lambda/handler.ts#L108

@floydspace
Copy link
Owner Author

Here is how Hono handles it

I just checked the Hono implementation, it looks good, I might take inspiration from it, the implementation in this PR is based on serverless-express, and it supports more events than hono, but I don't think all the events are usefull..
thanks for the reference

@floydspace floydspace force-pushed the feat/83-http-api-lambda branch from 41c88a7 to c9f496c Compare April 25, 2025 00:06
@floydspace
Copy link
Owner Author

Looking through the code, one thing that might be missing are Lambda Function URL events

import type { LambdaFunctionURLEvent } from "aws-lambda";

it is defined as type LambdaFunctionURLEvent = APIGatewayProxyEventV2 in @types/aws-lambda so it is actually covered

@floydspace floydspace marked this pull request as ready for review April 27, 2025 00:52
@floydspace floydspace merged commit f3c17c8 into main Apr 28, 2025
5 checks passed
@github-actions github-actions bot mentioned this pull request Apr 28, 2025
@godu godu mentioned this pull request Jul 21, 2025
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