-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update manifest and endpoints for the V3 of the appsec login events #3730
Conversation
986752c
to
b7055b2
Compare
b7055b2
to
e5ac522
Compare
a91fed2
to
86f64c7
Compare
Be careful, if any test logic is modified, you must not use any language prefix in PR title. Otherwise, other langs won't be tested, and you take the risk of breaking other repo's CI |
86f64c7
to
3b73898
Compare
Yes, I wasn't sure about it (I did change the title a couple of times). I did update the branch and rerun the tests, the PHP failures seem to be unrelated to this PR. |
rebasing the branch...the PHP errors should disappear |
Yep, many thanks, now the PR is green 😄 |
Motivation
The new RFC joining all previous documents introduces several updates that we must test in the tracers.
Changes
Enable V3 of the automated login events tests and also update java weblogs to support all the new functionality like the signup events.
Workflow
codeowners
file quickly.🚀 Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist
[<language>]
, double-check that only<language>
is impacted by the changebuild-XXX-image
label is present