axiomhq / axiom-lambda-extension

Ingest logs and platform events from your lambda functions
14 stars 2 forks source link

Review before launch #2

Closed dasfmi closed 1 year ago

dasfmi commented 1 year ago

@dominicchapman @a-khaledf hey guys, since we still working on debugging the rust implementation and we don't have rust experts who has free time at the moment, I suggest we lunch the golang one since me and @a-khaledf can work on maintaining it, the rust version could be the next version when we have the resources to fix the bug that we found.

I have made some changes here, but there is an error that Abdelrahman mentioned, we still need to investigate that one:

"error":"Post \"https://app.dev.axiomtestlabs.co/api/v1/datasets/axiom-lambda-extension-go-2/ingest\": io: read/write on closed pipe"} 
a-khaledf commented 1 year ago

we will need to enhance the documents the same way that the rust version does, by adding lambda object with name and stuff. so that the filters work