Skip to content

Websockets 2695 v17#10409

Closed
catenacyber wants to merge 10 commits into
OISF:masterfrom
catenacyber:websockets-2695-v17
Closed

Websockets 2695 v17#10409
catenacyber wants to merge 10 commits into
OISF:masterfrom
catenacyber:websockets-2695-v17

Conversation

@catenacyber
Copy link
Copy Markdown
Contributor

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

Describe changes:

SV_BRANCH=pr/1571

OISF/suricata-verify#1571

#10375 rebased

@catenacyber catenacyber marked this pull request as ready for review February 13, 2024 21:07
@suricata-qa
Copy link
Copy Markdown

Information:

ERROR: QA failed on SURI_TLPW2_autofp_suri_time.

field baseline test %
SURI_TLPW2_autofp_stats_chk
.uptime 101 111 109.9%
SURI_TLPR1_stats_chk
.tcp.pseudo 2810 3014 107.26%

Pipeline 18444

catenacyber and others added 10 commits February 14, 2024 21:43
if no config option is found,
as is done for udp

Ticket: 6304
Including the one for websocket over HTTP/2
port is used in AppLayerProtoDetectProbingParserPort
and not in AppLayerProtoDetectProbingParserElement
As for WebSocket which is detected only by protocol change.
When there is a protocol change, and a specific protocol is
expected, like WebSeocket, always run it, no matter the port.
@suricata-qa
Copy link
Copy Markdown

Information:

ERROR: QA failed on SURI_TLPW2_autofp_suri_time.

field baseline test %
SURI_TLPW2_autofp_stats_chk
.uptime 101 111 109.9%
SURI_TLPR1_stats_chk
.tcp.pseudo 2810 3014 107.26%

Pipeline 18562

@catenacyber catenacyber added the needs rebase Needs rebase to main label Mar 1, 2024
@catenacyber
Copy link
Copy Markdown
Contributor Author

Rebased in #10842

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

Labels

needs rebase Needs rebase to main

Development

Successfully merging this pull request may close these issues.

2 participants