Skip to content

Commit

Permalink
Fix CEF processor mapping exception (elastic#17220)
Browse files Browse the repository at this point in the history
Extensions spriv/dpriv were mapped to {source|destination}.user.group
instead of {source|destination}.user.group.name.

This caused a mapping exception when attempting to ingest CEF logs that
used those extensions.

Fixes elastic#17216

(cherry picked from commit 99f483e)
  • Loading branch information
adriansr committed Mar 24, 2020
1 parent 60b14fa commit af77fe3
Show file tree
Hide file tree
Showing 4 changed files with 33 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.next.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d

- Fix s3 input with cloudtrail fileset reading json file. {issue}16374[16374] {pull}16441[16441]
- Add queue_url definition in manifest file for aws module. {pull}16640{16640}
- Fixed a mapping exception when ingesting CEF logs that used the spriv or dpriv extensions. {issue}17216[17216] {pull}17220[17220]

*Heartbeat*

Expand Down
1 change: 1 addition & 0 deletions x-pack/filebeat/module/cef/log/test/cef.log
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
CEF:0|Elastic|Vaporware|1.0.0-alpha|18|Web request|low|eventId=3457 requestMethod=POST slat=38.915 slong=-77.511 proto=TCP sourceServiceName=httpd requestContext=https://www.google.com src=6.7.8.9 spt=33876 dst=192.168.10.1 dpt=443 request=https://www.example.com/cart
CEF:0|Elastic|Vaporware|1.0.0-alpha|18|Authentication|low|spriv=user dpriv=root
29 changes: 29 additions & 0 deletions x-pack/filebeat/module/cef/log/test/cef.log-expected.json
Original file line number Diff line number Diff line change
Expand Up @@ -50,5 +50,34 @@
"cef"
],
"url.original": "https://www.example.com/cart"
},
{
"cef.device.event_class_id": "18",
"cef.device.product": "Vaporware",
"cef.device.vendor": "Elastic",
"cef.device.version": "1.0.0-alpha",
"cef.extensions.destinationUserPrivileges": "root",
"cef.extensions.sourceUserPrivileges": "user",
"cef.name": "Authentication",
"cef.severity": "low",
"cef.version": "0",
"destination.user.group.name": "root",
"event.code": "18",
"event.dataset": "cef.log",
"event.module": "cef",
"event.original": "CEF:0|Elastic|Vaporware|1.0.0-alpha|18|Authentication|low|spriv=user dpriv=root",
"event.severity": 0,
"fileset.name": "log",
"input.type": "log",
"log.offset": 269,
"message": "Authentication",
"observer.product": "Vaporware",
"observer.vendor": "Elastic",
"observer.version": "1.0.0-alpha",
"service.type": "cef",
"source.user.group.name": "user",
"tags": [
"cef"
]
}
]
4 changes: 2 additions & 2 deletions x-pack/filebeat/processors/decode_cef/keys.ecs.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ var ecsExtensionMapping = map[string]mappedField{
"destinationTranslatedPort": {Target: "destination.nat.port"},
"destinationUserId": {Target: "destination.user.id"},
"destinationUserName": {Target: "destination.user.name"},
"destinationUserPrivileges": {Target: "destination.user.group"},
"destinationUserPrivileges": {Target: "destination.user.group.name"},
"deviceAction": {Target: "event.action"},
"deviceAddress": {Target: "observer.ip"},
"deviceDirection": {
Expand Down Expand Up @@ -108,7 +108,7 @@ var ecsExtensionMapping = map[string]mappedField{
"sourceTranslatedPort": {Target: "source.nat.port"},
"sourceUserId": {Target: "source.user.id"},
"sourceUserName": {Target: "source.user.name"},
"sourceUserPrivileges": {Target: "source.user.group"},
"sourceUserPrivileges": {Target: "source.user.group.name"},
"startTime": {Target: "event.start"},
"transportProtocol": {
Target: "network.transport",
Expand Down

0 comments on commit af77fe3

Please sign in to comment.