Project

General

Profile

Actions

Optimization #5545

open
JF CT

prefilter keyword: increase code coverage

Optimization #5545: prefilter keyword: increase code coverage

Added by Juliana Fajardini Reichow over 3 years ago. Updated 5 months ago.

Status:
New
Priority:
Normal
Target version:
Effort:
Difficulty:
Label:
Outreachy

Description

  • This is an onboarding task for the Outreachy project for increasing code coverage for our rule keywords ***

We have many keywords that have the 'prefilter' feature, for which the prefilter keyword itself isn't yet tested.

This task encompasses the simpler cases, for which one will have to add the 'prefilter' keyword to an already existing suricata-verify test that covers the rule keyword (this would be the case for 'filemagic' for instance). Some cases would also require creating the SV tests themselves. Those will be covered by a different ticket.

Examples of tests for 'filemagic':
- filestore-filecontainer-http
- filestore-filecontainer-smb

How do we know that the prefilter for filemagic needs coverage? We check https://app.codecov.io/gh/OISF/suricata/blob/master/src/detect-filemagic.c

(An example of a case that does not fit in this task is the fragbits keyword. While it supports 'prefilter', we must first add coverage to the 'fragbits' keyword itself: https://app.codecov.io/gh/OISF/suricata/blob/master/src/detect-fragbits.c)


Related issues 1 (1 open0 closed)

Related to Suricata - Documentation #5543: userguide: document which keywords accept the prefilter keywordAssignedVictor JulienActions

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

  • Related to Documentation #5543: userguide: document which keywords accept the prefilter keyword added

JF Updated by Juliana Fajardini Reichow over 3 years ago Actions #2

  • Description updated (diff)

JF Updated by Juliana Fajardini Reichow over 3 years ago Actions #3

  • Description updated (diff)

JF Updated by Juliana Fajardini Reichow over 3 years ago Actions #4

  • Target version changed from TBD to 8.0.0-beta1
  • Private changed from Yes to No

VJ Updated by Victor Julien almost 2 years ago Actions #5

  • Target version changed from 8.0.0-beta1 to TBD

OF Updated by Olusegun Fajobi 5 months ago Actions #6

  • Assignee changed from Community Ticket to Olusegun Fajobi

OF Updated by Olusegun Fajobi 5 months ago Actions #7

  • Assignee changed from Olusegun Fajobi to Community Ticket

OF Updated by Olusegun Fajobi 5 months ago Actions #8

  • Assignee changed from Community Ticket to Olusegun Fajobi

OF Updated by Olusegun Fajobi 5 months ago Actions #9

  • Status changed from New to Assigned

OF Updated by Olusegun Fajobi 5 months ago Actions #10

  • Assignee changed from Olusegun Fajobi to Community Ticket

OF Updated by Olusegun Fajobi 5 months ago Actions #11

  • Status changed from Assigned to In Progress

OF Updated by Olusegun Fajobi 5 months ago Actions #12

  • Status changed from In Progress to New
Actions

Also available in: PDF Atom