-
Notifications
You must be signed in to change notification settings - Fork 423
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
[Azure] Application Gateway WAF: add event.reason #10007
Conversation
(original)properties.details.data as rule.matched_data
💚 CLA has been signed |
This comment was marked as resolved.
This comment was marked as resolved.
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
/test |
🚀 Benchmarks reportPackage
|
Data stream | Previous EPS | New EPS | Diff (%) | Result |
---|---|---|---|---|
springcloudlogs |
4098.36 | 3115.26 | -983.1 (-23.99%) | 💔 |
application_gateway |
2967.36 | 1733.1 | -1234.26 (-41.59%) | 💔 |
auditlogs |
1945.53 | 1526.72 | -418.81 (-21.53%) | 💔 |
eventhub |
333333.33 | 250000 | -83333.33 (-25%) | 💔 |
firewall_logs |
1420.45 | 1113.59 | -306.86 (-21.6%) | 💔 |
graphactivitylogs |
1834.86 | 1451.38 | -383.48 (-20.9%) | 💔 |
identity_protection |
4608.29 | 2724.8 | -1883.49 (-40.87%) | 💔 |
platformlogs |
4950.5 | 2849 | -2101.5 (-42.45%) | 💔 |
provisioning |
2898.55 | 2288.33 | -610.22 (-21.05%) | 💔 |
To see the full report comment with /test benchmark fullreport
Quality Gate passedIssues Measures |
...stream/application_gateway/_dev/test/pipeline/test-application-gateway-raw.log-expected.json
Outdated
Show resolved
Hide resolved
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
Signed. |
/test |
@jH- could you please fix the merge conflicts? |
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a bunch of changes outside of the packages/azure directory that need to undone. I think they were the result of merge issue.
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deleted
a2b7ebb
to
af7f7f8
Compare
[git-generate] elastic-package -C packages/azure test pipeline -g -d application_gateway
Merge issues were corrected. Changeset looks fine now.
/test |
💚 Build Succeeded
History
|
Quality Gate passedIssues Measures |
Package azure - 1.17.0 containing this change is available at https://epr.elastic.co/search?package=azure |
Proposed commit message
Checklist
changelog.yml
file.How to test this PR locally