Skip to content

Commit

Permalink
Add header requirement for req ID
Browse files Browse the repository at this point in the history
  • Loading branch information
lahsivjar committed Nov 24, 2022
1 parent 963e361 commit 896ddd4
Showing 1 changed file with 39 additions and 23 deletions.
62 changes: 39 additions & 23 deletions module/apmlambda/lambda.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ package apmlambda // import "go.elastic.co/apm/module/apmlambda/v2"

import (
"bytes"
"fmt"
"log"
"net"
"net/http"
Expand Down Expand Up @@ -111,29 +112,8 @@ func (f *Function) Invoke(req *messages.InvokeRequest, response *messages.Invoke
lambdaContext.Request = formatPayload(req.Payload)
lambdaContext.Response = ""

if !ignoreTxnRegistration {
defer jsonw.Reset()
if err := createPartialTransactionJSON(tx, &jsonw); err != nil {
log.Printf("failed to create partial transaction for registration: %v", err)
} else {
resp, err := http.Post(
// TODO: @lahsivjar better way to get base URI
"http://localhost:8200/register/transaction",
"application/vnd.elastic.apm.transaction+json",
bytes.NewReader(jsonw.Bytes()),
)
// Don't attempt registration for next invocations if network
// error or the registration endpoint is not found.
if err != nil || resp.StatusCode == 404 {
ignoreTxnRegistration = true
}
if err != nil {
log.Printf("failed to register transaction, req failed with error: %v", err)
}
if resp.StatusCode/100 != 2 {
log.Printf("failed to register transaction, req failed with status code: %d", resp.StatusCode)
}
}
if err := registerTxn(tx, req.RequestId); err != nil {
log.Printf("failed to register txn: %v", err)
}
err := f.client.Call("Function.Invoke", req, response)
if err != nil {
Expand All @@ -154,6 +134,42 @@ func (f *Function) Invoke(req *messages.InvokeRequest, response *messages.Invoke
return nil
}

func registerTxn(tx *apm.Transaction, requestID string) error {
if ignoreTxnRegistration {
return nil
}

defer jsonw.Reset()
if err := createPartialTransactionJSON(tx, &jsonw); err != nil {
return fmt.Errorf("failed to create txn registration body: %v", err)
}
req, err := http.NewRequest(
http.MethodPost,
// TODO: @lahsivjar better way to get base URI
"http://localhost:8200/register/transaction",
bytes.NewReader(jsonw.Bytes()),
)
if err != nil {
return fmt.Errorf("failed to create txn registration request: %v", err)
}
req.Header.Set("Content-Type", "application/vnd.elastic.apm.transaction+json")
req.Header.Set("x-elastic-aws-request-id", requestID)

resp, err := http.DefaultClient.Do(req)
// Don't attempt registration for next invocations if network
// error or the registration endpoint is not found.
if err != nil || resp.StatusCode == 404 {
ignoreTxnRegistration = true
}
if err != nil {
return fmt.Errorf("failed to register transaction, req failed with error: %v", err)
}
if resp.StatusCode/100 != 2 {
return fmt.Errorf("failed to register transaction, req failed with status code: %d", resp.StatusCode)
}
return nil
}

type invokeResponseError struct {
err *messages.InvokeResponse_Error
}
Expand Down

0 comments on commit 896ddd4

Please sign in to comment.