Skip to content
This repository was archived by the owner on May 21, 2025. It is now read-only.

chiadapter: support API Gateway Events V2 #136

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

iamsalnikov
Copy link
Contributor

*Issue #134 *

Description of changes:

I've implemented the support of Gateway Events V2 in chiadapter. I am not sure if everything is ok with the style you prefer, please, feel free to review it and I'll fix all the issues :)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@iamsalnikov
Copy link
Contributor Author

I've fixed failing tests in #137

@iamsalnikov
Copy link
Contributor Author

Failing tests are fixed in #131

@iamsalnikov
Copy link
Contributor Author

I've synced the branch with master to fix the tests.

@iamsalnikov
Copy link
Contributor Author

Can anyone review and merge it if everything is ok?

cc @sapessi, @drakejin

@zekth
Copy link

zekth commented Nov 8, 2022

Can we land this please? @sapessi @drakejin ?

@kateile
Copy link

kateile commented Apr 6, 2023

Can we merge this

@miki725
Copy link

miki725 commented May 25, 2023

as lambda urls only work with v2 format, this would be very useful if can be merged

https://docs.aws.amazon.com/lambda/latest/dg/urls-invocation.html#urls-payloads

@drakejin
Copy link
Contributor

drakejin commented May 25, 2023

Oh my god. I am a maintainer! I didn't notice!

Copy link
Contributor

@drakejin drakejin left a comment

Choose a reason for hiding this comment

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

Wait, I'll check it

@drakejin
Copy link
Contributor

#103
#136
#156
#163

@sapessi It's the same requirement. You should check these PRs... I am also a 'chi' user. I need this.

chi users are waiting for a long time.

@sp-pau-tena
Copy link

@drakejin any news about when this is going to be merged?

@aleksasiriski
Copy link

aleksasiriski commented Mar 31, 2024

Please review and merge this, it's much needed.

This is already supported? https://github.com/awslabs/aws-lambda-go-api-proxy/blob/master/chi/adapterv2.go

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants