Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

output: Add linktype description #10956

Closed
wants to merge 1 commit into from
Closed

Conversation

jlucovsky
Copy link
Contributor

Amend the linktype output with the linktype name (when available).

The linktype name is included alongside linktype when alert.packet is enabled. The name is retrieved from pcap_datalink_val_to_name with the thought being that the pcap library in use can also translate the linktype value into a descriptive value.

Link to redmine ticket: 6954

Describe changes:

  • Include the linktype name alongside linktype
  • Update the schema with linktype_name

Provide values to any of the below to override the defaults.

SV_BRANCH=OISF/suricata-verify#1798

Amend the linktype output with the linktype name (when available).

The linktype name is included alongside linktype when `alert.packet` is
enabled. The name is retrieved from pcap_datalink_val_to_name with the
thought being that the pcap library in use can also translate the
linktype value into a descriptive value.
@jlucovsky jlucovsky requested review from victorjulien and a team as code owners April 25, 2024 14:31
Copy link

codecov bot commented Apr 25, 2024

Codecov Report

Attention: Patch coverage is 66.66667% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 82.93%. Comparing base (ad4185b) to head (52ec372).

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #10956      +/-   ##
==========================================
- Coverage   82.94%   82.93%   -0.01%     
==========================================
  Files         918      918              
  Lines      248784   248787       +3     
==========================================
- Hits       206351   206340      -11     
- Misses      42433    42447      +14     
Flag Coverage Δ
fuzzcorpus 64.29% <66.66%> (+<0.01%) ⬆️
suricata-verify 62.65% <66.66%> (-0.02%) ⬇️
unittests 62.28% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@jlucovsky
Copy link
Contributor Author

Closing for reevaluation.

@jlucovsky jlucovsky closed this Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant