Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
💚 Flaky test reportTests succeeded. 🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
|
@eyalkoren Your changes LGTM! |
...ambda-plugin/src/main/java/co/elastic/apm/agent/awslambda/RequestHandlerInstrumentation.java
Outdated
Show resolved
Hide resolved
...src/main/java/co/elastic/apm/agent/awslambda/helper/AbstractAPIGatewayTransactionHelper.java
Outdated
Show resolved
Hide resolved
eyalkoren
left a comment
There was a problem hiding this comment.
Tested manually:
- all handler types, with the matching event type and with a non-matching event type
- both options of handle method specification -
package.Handlervspackage.Handler::handleRequest - metadata
- transaction context and name
- some special configurations (e.g.
log_levelandtrace_methods)
Documents were added as well, and an additional release step (not automated atm as it is probably only temporary).
❕ Build Aborted
Expand to view the summary
Build stats
Test stats 🧪
🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
What does this PR do?
Closes #1911
Checklist