slim icon indicating copy to clipboard operation
slim copied to clipboard

[WIP]inital step for lambdaproxy probe

Open mritunjaysharma394 opened this issue 2 years ago • 1 comments

Signed-off-by: Mritunjay Sharma [email protected]

mritunjaysharma394 avatar May 19 '22 14:05 mritunjaysharma394

👇 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

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

ghost avatar May 19 '22 14:05 ghost

@mritunjaysharma394 the test data seems ok. what's next here?

kcq avatar Aug 21 '22 21:08 kcq

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

mritunjaysharma394 avatar Aug 22 '22 08:08 mritunjaysharma394