Skip to content

Pickle - reconstructs the packet from field valies, payload and metadata#4919

Open
guedou wants to merge 1 commit intosecdev:masterfrom
guedou:guedou/issue_4599
Open

Pickle - reconstructs the packet from field valies, payload and metadata#4919
guedou wants to merge 1 commit intosecdev:masterfrom
guedou:guedou/issue_4599

Conversation

@guedou
Copy link
Member

@guedou guedou commented Feb 15, 2026

This PR attempts to fix #4599 but introduces a breaking change as packets used to be built before being pickled. The new code recreates a packet from field values, payload and metadata.

@guedou guedou requested a review from gpotter2 February 15, 2026 13:22
@codecov
Copy link

codecov bot commented Feb 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.64%. Comparing base (f303033) to head (b9f54f2).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4919      +/-   ##
==========================================
- Coverage   80.84%   80.64%   -0.20%     
==========================================
  Files         369      370       +1     
  Lines       90968    91737     +769     
==========================================
+ Hits        73544    73985     +441     
- Misses      17424    17752     +328     
Files with missing lines Coverage Δ
scapy/packet.py 84.76% <100.00%> (+0.10%) ⬆️

... and 20 files with indirect coverage changes

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pickle changes/sets payload_length of DoIP package

1 participant