Skip to content

Comments

doc: upgrade guide for logging http custom headers#8978

Closed
catenacyber wants to merge 2 commits intoOISF:masterfrom
catenacyber:http-customlog-5320-v7.1
Closed

doc: upgrade guide for logging http custom headers#8978
catenacyber wants to merge 2 commits intoOISF:masterfrom
catenacyber:http-customlog-5320-v7.1

Conversation

@catenacyber
Copy link
Contributor

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/5320

Describe changes:

@catenacyber catenacyber requested a review from jufajardini as a code owner June 6, 2023 19:56
@catenacyber catenacyber added the typo/doc update No code change : only doc or typo fixes label Jun 6, 2023
@codecov
Copy link

codecov bot commented Jun 6, 2023

Codecov Report

Merging #8978 (d1ee18c) into master (6154bab) will decrease coverage by 0.02%.
The diff coverage is n/a.

❗ Current head d1ee18c differs from pull request most recent head 7784f15. Consider uploading reports for the commit 7784f15 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8978      +/-   ##
==========================================
- Coverage   82.42%   82.41%   -0.02%     
==========================================
  Files         969      969              
  Lines      273476   273476              
==========================================
- Hits       225410   225376      -34     
- Misses      48066    48100      +34     
Flag Coverage Δ
fuzzcorpus 64.93% <ø> (-0.01%) ⬇️
suricata-verify 60.48% <ø> (-0.03%) ⬇️
unittests 62.92% <ø> (+<0.01%) ⬆️

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

Copy link
Contributor

@jufajardini jufajardini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just two minor suggestions. :)

Co-authored-by: Juliana Fajardini Reichow <jufajardini@gmail.com>
@catenacyber
Copy link
Contributor Author

Replaced by #8991

@catenacyber catenacyber closed this Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

typo/doc update No code change : only doc or typo fixes

Development

Successfully merging this pull request may close these issues.

2 participants