Project

General

Profile

Actions

Bug #7332

closed
JF JF

tls: fix duplicate EVE field issuerdn

Bug #7332: tls: fix duplicate EVE field issuerdn

Added by Juliana Fajardini Reichow over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Target version:
Affected Versions:
Effort:
Difficulty:
Label:

Description

When enabling the field subjectaltname in custom logging, the JSON schema validator complains that the issuerdn key
is duplicated:

tls-eve-custom-fields/output/eve.json: duplicate key error: key=issuerdn, current value=C=FR, ST=IDF, L=Paris, O=Stamus, CN=SELKS, new value=C=FR, ST=IDF, L=Paris, O=Stamus, CN=SELKS

(output from modified version of SV test using ja4-tls test's pcap)

Relevant YAML config:

outputs:¬
  - eve-log:¬
      enabled: yes¬
      types:¬
        - tls:¬
            extended: yes     # enable this for extended logging information¬
            custom: [subject, issuer, subjectaltname]¬

TLS event (stripped of non-tls fields for readability):

{
  "event_type":"tls",
  "pkt_src":"wire/pcap",
  "tls":{
     "subject":"C=FR, ST=IDF, L=Paris, O=Stamus, CN=SELKS",
     "issuerdn":"C=FR, ST=IDF, L=Paris, O=Stamus, CN=SELKS",
     "issuerdn":"C=FR, ST=IDF, L=Paris, O=Stamus, CN=SELKS" 
  }
}

Investigation indicates that when logging the subject alternative name in custom logging, we were actually calling the issuerdn logging function.

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #1

  • Description updated (diff)

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #2

  • Description updated (diff)

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #3

  • Subject changed from tls: duplicate EVE field issuerdn to tls: fix :wqduplicate EVE field issuerdn

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #4

  • Subject changed from tls: fix :wqduplicate EVE field issuerdn to tls: fix duplicate EVE field issuerdn

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #5

  • Description updated (diff)

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #6

  • Status changed from New to In Review

VJ Updated by Victor Julien over 1 year ago Actions #7

Does 7 have this issue as well?

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #8

Checked and didn't see it, as log is done differently, but will double-check.

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #9

Victor Julien wrote in #note-7:

Does 7 have this issue as well?

subjectaltname was introduced in 8, only: https://redmine.openinfosecfoundation.org/issues/5234

JF Updated by Juliana Fajardini Reichow over 1 year ago Actions #10

  • Status changed from In Review to Closed
Actions

Also available in: PDF Atom