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
[AWS] Support Cloudtrail tlsDetails field #6352
Conversation
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
bump |
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as |
Hi! This PR has been stale for a while and we're going to close it as part of our cleanup procedure. We appreciate your contribution and would like to apologize if we have not been able to review it, due to the current heavy load of the team. Feel free to re-open this PR if you think it should stay open and is worth rebasing. Thank you for your contribution! |
I haven't had the time to follow up on this or push it along. It should be
GTG, might need a rebase.
…On Tue, Sep 26, 2023 at 4:35 PM kaiyan-sheng ***@***.***> wrote:
Reopened #6352 <#6352>.
—
Reply to this email directly, view it on GitHub
<#6352 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADEEL4BKWPEDLFRV3LKUQO3X4NDAVANCNFSM6AAAAAAYSAILEM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
/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.
LGTM after the builder can be made to run and pass.
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.
Thank you for making the change!
🌐 Coverage report
|
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.
Still LGTM
Package aws - 2.5.0 containing this change is available at https://epr.elastic.co/search?package=aws |
What does this PR do?
Parses tlsDetails field
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots