Skip to content

10 sequential reads of SSL should only cause us to bomb out if all re…#3187

Draft
davehorton wants to merge 2 commits into
warmcat:mainfrom
jambonz:fix/ws-client-sanity-fix
Draft

10 sequential reads of SSL should only cause us to bomb out if all re…#3187
davehorton wants to merge 2 commits into
warmcat:mainfrom
jambonz:fix/ws-client-sanity-fix

fix prev commit

fc2aba6
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jun 16, 2026 in 4m 45s

Quality Gate failed

Failed conditions
D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check failure on line 1239 in lib/roles/ws/ops-ws.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest more than 3 if|for|do|while|switch statements.

See more on https://sonarcloud.io/project/issues?id=warmcat_libwebsockets&issues=AZ7QMffx-5iz9GRiI4BY&open=AZ7QMffx-5iz9GRiI4BY&pullRequest=3187

Check failure on line 1270 in lib/roles/ws/ops-ws.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 47 to the 25 allowed.

See more on https://sonarcloud.io/project/issues?id=warmcat_libwebsockets&issues=AZ7QMffx-5iz9GRiI4Ba&open=AZ7QMffx-5iz9GRiI4Ba&pullRequest=3187

Check warning on line 953 in lib/roles/ws/ops-ws.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define each identifier in a dedicated statement.

See more on https://sonarcloud.io/project/issues?id=warmcat_libwebsockets&issues=AZ7QMffx-5iz9GRiI4BZ&open=AZ7QMffx-5iz9GRiI4BZ&pullRequest=3187