Skip to content

Events: R, W, RW -> messages migration - #242

Merged
nt0xa merged 1 commit into
mainfrom
refactor/events-data-fields
Jul 12, 2026
Merged

Events: R, W, RW -> messages migration#242
nt0xa merged 1 commit into
mainfrom
refactor/events-data-fields

Conversation

@nt0xa

@nt0xa nt0xa commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Before interaction data was stored in R, W and RW fields, now it is stored in one Data field. Data is array of []byte, every element corresponds to protocol message (request/response for http, dns query/answer for dns, all commands for ftp/smtp)

Before interaction data was stored in R, W and RW fields, now it is stored in one Data field. Data is array of []byte, every element corresponds to protocol message (request/response for http, dns query/answer for dns, all commands for ftp/smtp)
@nt0xa nt0xa added refactor Change that neither fixes a bug or adds a feature minor Minor changes labels Jul 12, 2026
@codecov

codecov Bot commented Jul 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 20.89552% with 53 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.36%. Comparing base (aaea3bf) to head (ba5ac1f).

Files with missing lines Patch % Lines
pkg/netx/logging.go 0.00% 16 Missing ⚠️
internal/cmd/server/events.go 0.00% 6 Missing ⚠️
internal/modules/slack/notifier.go 0.00% 5 Missing ⚠️
internal/cmd/server/dns.go 0.00% 4 Missing ⚠️
internal/cmd/server/ftp.go 0.00% 4 Missing ⚠️
internal/cmd/server/http.go 0.00% 4 Missing ⚠️
internal/cmd/server/smtp.go 0.00% 4 Missing ⚠️
internal/service/dbsvc/event.go 0.00% 4 Missing ⚠️
internal/modules/lark/notifier.go 0.00% 3 Missing ⚠️
internal/modules/telegram/notifier.go 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #242      +/-   ##
==========================================
- Coverage   42.48%   42.36%   -0.13%     
==========================================
  Files         213      213              
  Lines        8067     8057      -10     
==========================================
- Hits         3427     3413      -14     
- Misses       4362     4367       +5     
+ Partials      278      277       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nt0xa
nt0xa merged commit 3c3bf29 into main Jul 12, 2026
2 of 4 checks passed
@nt0xa
nt0xa deleted the refactor/events-data-fields branch July 12, 2026 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Minor changes refactor Change that neither fixes a bug or adds a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant