Skip to content

Changes network to use ECS fields#44392

Merged
FrankHassanabad merged 2 commits intoelastic:masterfrom
FrankHassanabad:change-dns-fields-ecs
Aug 30, 2019
Merged

Changes network to use ECS fields#44392
FrankHassanabad merged 2 commits intoelastic:masterfrom
FrankHassanabad:change-dns-fields-ecs

Conversation

@FrankHassanabad
Copy link
Copy Markdown
Contributor

@FrankHassanabad FrankHassanabad commented Aug 29, 2019

Summary

Change dns.question.etld_plus_one to dns.question.registered_domain in the query and the draggable.

You should see on the network page/Top DNS domains this change:
drag-and-drop-table

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/siem

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Copy link
Copy Markdown
Contributor

@XavierM XavierM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Happy ECS

"dns.question.type",
"dns.question.class",
"dns.question.etld_plus_one",
"dns.question.registered_domain",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That worked, I always tried to modify JSON and I failed each time. Do you have a technique?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still modified the JSON.

I ran:

node ../scripts/es_archiver edit packetbeat/dns

And then edited it, closed it and then re-pushed.

@FrankHassanabad FrankHassanabad changed the title Change network to use ECS fields Changes network to use ECS fields Aug 30, 2019
@FrankHassanabad FrankHassanabad merged commit 7a0a3f5 into elastic:master Aug 30, 2019
@FrankHassanabad FrankHassanabad deleted the change-dns-fields-ecs branch August 30, 2019 20:33
FrankHassanabad added a commit to FrankHassanabad/kibana that referenced this pull request Aug 30, 2019
* Change network to use ECS fields

* Updated functional tests with new field mapping and data
FrankHassanabad added a commit to FrankHassanabad/kibana that referenced this pull request Aug 30, 2019
* Change network to use ECS fields

* Updated functional tests with new field mapping and data
FrankHassanabad added a commit that referenced this pull request Aug 30, 2019
* Change network to use ECS fields

* Updated functional tests with new field mapping and data
FrankHassanabad added a commit that referenced this pull request Aug 30, 2019
* Change network to use ECS fields

* Updated functional tests with new field mapping and data
jloleysens added a commit to jloleysens/kibana that referenced this pull request Sep 2, 2019
…-to-np-ready

* 'master' of github.com:elastic/kibana: (409 commits)
  [ML] Data frame analytics: Fix source index checks. (elastic#44479)
  [Code] try fix this test (elastic#43692)
  skip flaky suite (elastic#44572) (elastic#42111) (elastic#44286) (elastic#43557) (elastic#42567)
  skip flaky suite (elastic#44560)
  skip flaky suite (elastic#44250)
  disable flaky suite (elastic#41336)
  [code] Update download URLs for go lsp. (elastic#44581)
  disable flaky suite (elastic#44575)
  disable flaky suite (elastic#44576)
  [Code] add functional test to verify lang server full api symbol/reference counts (elastic#44051)
  Improve Storybook scripts and load time (elastic#44511)
  Update Dependencies (elastic#44519)
  Remove use of injecti18n in Embeddables plugin (elastic#44043)
  [SIEM] Adds a configuraton option for the default SIEM date time range (elastic#44540)
  [Uptime]Fix/issue 40584  section headline should be inside panel (elastic#43468)
  disable flaky suite (elastic#22322)
  Changes network to use ECS fields (elastic#44392)
  Fix 'workpad flash' when loading new workpad (elastic#44387)
  [renovate] bump new PR version labels
  Update dependency cmd-shim to ^2.1.0 (elastic#44034)
  ...

# Conflicts:
#	src/legacy/core_plugins/console/public/index.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants