Skip to content

Commit

Permalink
[Uptime] Fix nesting of x509 subject
Browse files Browse the repository at this point in the history
In elastic#17687 we added additional x509 fields. The mapping accidentally
double nested subject fields as subject.subject. This fixes that.

There are no tests here because we don't really have testing around
mappings, and it's sort of difficult to test without being repetitive,
so none are included here.
  • Loading branch information
andrewvc committed Apr 30, 2020
1 parent bff78c9 commit 6a4d46b
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions heartbeat/docs/fields.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -8019,7 +8019,7 @@ example: SHA256-RSA
--
*`tls.server.x509.subject.subject.common_name`*::
*`tls.server.x509.subject.common_name`*::
+
--
List of common names (CN) of subject.
Expand All @@ -8030,14 +8030,14 @@ example: r2.shared.global.fastly.net
--
*`tls.server.x509.subject.subject.common_name.text`*::
*`tls.server.x509.subject.common_name.text`*::
+
--
type: text
--
*`tls.server.x509.subject.subject.distinguished_name`*::
*`tls.server.x509.subject.distinguished_name`*::
+
--
Distinguished name (DN) of the certificate subject entity.
Expand Down
2 changes: 1 addition & 1 deletion heartbeat/include/fields.go

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions heartbeat/monitors/active/dialchain/_meta/fields.yml
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@
- name: subject
type: group
fields:
- name: subject.common_name
- name: common_name
type: keyword
ignore_above: 1024
description: List of common names (CN) of subject.
Expand All @@ -150,7 +150,7 @@
- name: text
type: text
analyzer: simple
- name: subject.distinguished_name
- name: distinguished_name
type: keyword
ignore_above: 1024
description: Distinguished name (DN) of the certificate subject entity.
Expand Down

0 comments on commit 6a4d46b

Please sign in to comment.