Skip to content

Fix LpClassName colllect bug in the RRP abstraction layer#4971

Closed
Ebrix wants to merge 2 commits intosecdev:masterfrom
Ebrix:master
Closed

Fix LpClassName colllect bug in the RRP abstraction layer#4971
Ebrix wants to merge 2 commits intosecdev:masterfrom
Ebrix:master

Conversation

@Ebrix
Copy link
Copy Markdown
Contributor

@Ebrix Ebrix commented Apr 15, 2026

Checklist:

  • If you are new to Scapy: I have checked CONTRIBUTING.md (esp. section submitting-pull-requests)
  • I squashed commits belonging together
  • I added unit tests or explained why they are not relevant

Unit tests are done in scapy-red

  • I executed the regression tests (using tox)

In the event of a ClassName set on a registry entry, the PR make it so the function does a second roundtrip with the RPC to actually collect the value.

@Ebrix Ebrix changed the title Fix bug in RRP abstraction layer Fix LpClassName colllect bug in the RRP abstraction layer Apr 15, 2026
@Ebrix Ebrix closed this Apr 15, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.32%. Comparing base (5009233) to head (ab42766).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4971   +/-   ##
=======================================
  Coverage   80.31%   80.32%           
=======================================
  Files         381      381           
  Lines       93630    93630           
=======================================
+ Hits        75202    75206    +4     
+ Misses      18428    18424    -4     

see 8 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.

1 participant