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
all: document valid time duration units #6992
Conversation
🌐 Coverage report
|
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
/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, just a couple minor nits:
- Some descriptions only specify the following units: "h, m, s", while others list "ns, us, ms, s, m, h"
- Some descriptions list "h/m/s" while others list "h, m, s"
Agreed on the absence of a consistent convention. I (generally) followed package-local convention when adding the changes. We should decide on a consistent convention for all packages and then resolve to that, but this is work for a future PR IMO. The first issue is intentional, while nearly all durations are parsed via |
f6ac0a7
to
8b04145
Compare
Also harmonise docs and clean up some copy/paste errors.
Sorry for the late response, didn't see the notification for your comment. Definitely agree, it'd be better to discuss it later and handle in a separate PR (this one is already quite big in terms of the number of files being touched). That makes sense for the first issue. I was blindly assuming that everything consumed the Go duration, but makes sense that we'd extract some value from it to feed to a service. |
Package 1password - 1.16.0 containing this change is available at https://epr.elastic.co/search?package=1password |
Package akamai - 2.13.0 containing this change is available at https://epr.elastic.co/search?package=akamai |
Package atlassian_bitbucket - 1.13.0 containing this change is available at https://epr.elastic.co/search?package=atlassian_bitbucket |
Package atlassian_confluence - 1.14.0 containing this change is available at https://epr.elastic.co/search?package=atlassian_confluence |
Package atlassian_jira - 1.14.0 containing this change is available at https://epr.elastic.co/search?package=atlassian_jira |
Package bitwarden - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=bitwarden |
Package carbon_black_cloud - 1.12.0 containing this change is available at https://epr.elastic.co/search?package=carbon_black_cloud |
Package infoblox_bloxone_ddi - 1.7.0 containing this change is available at https://epr.elastic.co/search?package=infoblox_bloxone_ddi |
Package lastpass - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=lastpass |
Package m365_defender - 1.15.0 containing this change is available at https://epr.elastic.co/search?package=m365_defender |
Package microsoft_exchange_online_message_trace - 1.9.0 containing this change is available at https://epr.elastic.co/search?package=microsoft_exchange_online_message_trace |
Package netflow - 2.11.0 containing this change is available at https://epr.elastic.co/search?package=netflow |
Package network_traffic - 1.20.0 containing this change is available at https://epr.elastic.co/search?package=network_traffic |
Package okta - 1.25.0 containing this change is available at https://epr.elastic.co/search?package=okta |
Package panw_cortex_xdr - 1.14.0 containing this change is available at https://epr.elastic.co/search?package=panw_cortex_xdr |
Package ping_one - 1.5.0 containing this change is available at https://epr.elastic.co/search?package=ping_one |
Package rapid7_insightvm - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=rapid7_insightvm |
Package sentinel_one_cloud_funnel - 0.2.0 containing this change is available at https://epr.elastic.co/search?package=sentinel_one_cloud_funnel |
Package slack - 1.8.0 containing this change is available at https://epr.elastic.co/search?package=slack |
Package snyk - 1.12.0 containing this change is available at https://epr.elastic.co/search?package=snyk |
Package sophos_central - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=sophos_central |
Package tanium - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=tanium |
Package tcp - 1.11.0 containing this change is available at https://epr.elastic.co/search?package=tcp |
Package tenable_io - 1.2.0 containing this change is available at https://epr.elastic.co/search?package=tenable_io |
Package ti_abusech - 1.16.0 containing this change is available at https://epr.elastic.co/search?package=ti_abusech |
Package ti_cif3 - 1.2.0 containing this change is available at https://epr.elastic.co/search?package=ti_cif3 |
Package ti_cybersixgill - 1.17.0 containing this change is available at https://epr.elastic.co/search?package=ti_cybersixgill |
Package ti_misp - 1.19.0 containing this change is available at https://epr.elastic.co/search?package=ti_misp |
Package ti_otx - 1.13.0 containing this change is available at https://epr.elastic.co/search?package=ti_otx |
Package ti_recordedfuture - 1.13.0 containing this change is available at https://epr.elastic.co/search?package=ti_recordedfuture |
Package ti_threatq - 1.14.0 containing this change is available at https://epr.elastic.co/search?package=ti_threatq |
Package trellix_epo_cloud - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=trellix_epo_cloud |
Package trend_micro_vision_one - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=trend_micro_vision_one |
Package udp - 1.11.0 containing this change is available at https://epr.elastic.co/search?package=udp |
Package zerofox - 1.14.0 containing this change is available at https://epr.elastic.co/search?package=zerofox |
Package zeronetworks - 1.2.0 containing this change is available at https://epr.elastic.co/search?package=zeronetworks |
Also harmonise docs and clean up some copy/paste errors.
What does this PR do?
See title.
Also harmonise docs and clean up some copy/paste errors.
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots