slim
slim copied to clipboard
[WIP]inital step for lambdaproxy probe
Signed-off-by: Mritunjay Sharma [email protected]
👇 Click on the image for a new way to code review
-
Make big changes easier — review code in small groups of related files
-
Know where to start — see the whole change at a glance
-
Take a code tour — explore the change with an interactive tour
-
Make comments and review — all fully sync’ed with github
Legend
@mritunjaysharma394 the test data seems ok. what's next here?
I think then it can be good to merge for now? However, the next step or probably the next improvement in my opinion will be to find a way to map the three structs, that is, HTTPProbeCmd
, HTTPRequest
and apiGatewayProxyRequest
more properly, currently we were able to map only three common fields in all three as found here
What and how we can map other fields among the three can be good step