Project

General

Profile

Actions

Bug #6304

closed
PA PA

schema.json : if protocol such as ENIP is detection only, we do not have _tcp suffix in stats

Bug #6304: schema.json : if protocol such as ENIP is detection only, we do not have _tcp suffix in stats

Added by Philippe Antoine over 2 years ago. Updated over 2 years ago.

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

Description

Meaning we get

1./stats/app_layer/error Additional properties are not allowed ('enip' was unexpected)
2./stats/app_layer/flow Additional properties are not allowed ('enip' was unexpected)
3./stats/app_layer/tx Additional properties are not allowed ('enip' was unexpected)

when running suricata-verify

Should we duplicate every protocol that can be over both TCP and UDP to have a third line in schema.json about it ?

PA Updated by Philippe Antoine over 2 years ago Actions #1

cf AppLayerSetupCounters

JI Updated by Jason Ish over 2 years ago Actions #2

Is there a valid reason why it doesn't have the suffix when in detection only mode? My feeling is there shouldn't be difference unless there is a good reason for it.

PA Updated by Philippe Antoine over 2 years ago Actions #3

I do not know of a reason but to have `_tcp` prefix, you need both detection and registration on both udp and tcp as per the code of `AppLayerSetupCounters`

PA Updated by Philippe Antoine over 2 years ago Actions #4

  • Status changed from New to In Review

PA Updated by Philippe Antoine over 2 years ago Actions #5

  • Priority changed from Low to Normal

PA Updated by Philippe Antoine over 2 years ago Actions #6

  • Assignee changed from OISF Dev to Philippe Antoine

PA Updated by Philippe Antoine over 2 years ago Actions #7

  • Status changed from In Review to Closed
Actions

Also available in: PDF Atom