scapy icon indicating copy to clipboard operation
scapy copied to clipboard

Add support for websockets

Open lucasdrufva opened this issue 1 year ago • 2 comments

Adds support for websockets.

HTTP dissector has been updated to support omitting the reason phrase as this is usually done when upgrading to websocket connection.

fixes #4578

lucasdrufva avatar Nov 03 '24 15:11 lucasdrufva

Hi ! This looks pretty cool !

gpotter2 avatar Nov 03 '24 17:11 gpotter2

Codecov Report

:x: Patch coverage is 98.07692% with 3 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 78.74%. Comparing base (8e08cbf) to head (60a13b7). :warning: Report is 176 commits behind head on master.

Files with missing lines Patch % Lines
scapy/contrib/websocket.py 97.97% 3 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4579      +/-   ##
==========================================
- Coverage   81.62%   78.74%   -2.89%     
==========================================
  Files         358      334      -24     
  Lines       85652    80602    -5050     
==========================================
- Hits        69915    63470    -6445     
- Misses      15737    17132    +1395     
Files with missing lines Coverage Δ
scapy/layers/http.py 80.54% <100.00%> (-1.90%) :arrow_down:
scapy/contrib/websocket.py 97.97% <97.97%> (ø)

... and 284 files with indirect coverage changes

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

codecov[bot] avatar Nov 06 '24 10:11 codecov[bot]