Skip to content

PgSQL native backend protocol: replace libpq on the data path (connect/auth/TLS, simple query, COPY, extended query via stmt pipeline, Describe cache) - #5882

Open
renecannao wants to merge 98 commits into
v3.0from
feature/pgsql-native-backend-protocol
Open

PgSQL native backend protocol: replace libpq on the data path (connect/auth/TLS, simple query, COPY, extended query via stmt pipeline, Describe cache)#5882
renecannao wants to merge 98 commits into
v3.0from
feature/pgsql-native-backend-protocol

Merge remote-tracking branch 'feature/pgsql-native-backend-protocol' …

6888955
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Sep 10, 2026 in 15m 45s

Quality Gate failed

Failed conditions
E Security 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 warning on line 2517 in lib/PgSQL_Connection.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZp&open=AaAUKVYFXMtocdt8EMZp&pullRequest=5882

Check warning on line 26 in test/tap/tests/unit/pgsql_backend_extq-t.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVLMXMtocdt8EMYV&open=AaAUKVLMXMtocdt8EMYV&pullRequest=5882

Check failure on line 233 in test/tap/tests/pgsql-native_prepared-t.cpp

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=sysown_proxysql&issues=AaAUKVOyXMtocdt8EMYt&open=AaAUKVOyXMtocdt8EMYt&pullRequest=5882

Check warning on line 2549 in lib/PgSQL_Connection.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZq&open=AaAUKVYFXMtocdt8EMZq&pullRequest=5882

Check failure on line 1580 in lib/PgSQL_Connection.cpp

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=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZU&open=AaAUKVYFXMtocdt8EMZU&pullRequest=5882

Check failure on line 3929 in lib/PgSQL_Connection.cpp

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=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZ3&open=AaAUKVYFXMtocdt8EMZ3&pullRequest=5882

Check warning on line 137 in test/tap/tests/unit/pgsql_backend_auth-t.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVCPXMtocdt8EMYJ&open=AaAUKVCPXMtocdt8EMYJ&pullRequest=5882

Check warning on line 30 in lib/PgSQL_Backend_Auth.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVUhXMtocdt8EMY2&open=AaAUKVUhXMtocdt8EMY2&pullRequest=5882

Check failure on line 2680 in lib/PgSQL_Protocol.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "new" with an operation that automatically manages the memory. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVcfXMtocdt8EMaQ&open=AaAUKVcfXMtocdt8EMaQ&pullRequest=5882

Check failure on line 3033 in lib/PgSQL_Connection.cpp

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=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZx&open=AaAUKVYFXMtocdt8EMZx&pullRequest=5882

Check failure on line 426 in lib/PgSQL_Thread.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

C-style cast removing const qualification from the type of a pointer may lead to undefined behavior. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVaNXMtocdt8EMZ_&open=AaAUKVaNXMtocdt8EMZ_&pullRequest=5882

Check failure on line 90 in test/tap/tests/unit/pgsql_stmt_meta_cache-t.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "new" with an operation that automatically manages the memory. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVMQXMtocdt8EMYZ&open=AaAUKVMQXMtocdt8EMYZ&pullRequest=5882

Check failure on line 4070 in lib/PgSQL_Connection.cpp

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=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZ6&open=AaAUKVYFXMtocdt8EMZ6&pullRequest=5882

Check failure on line 37 in lib/PgSQL_Backend_Auth.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly capture "out", "off" in a non-transient lambda. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVUhXMtocdt8EMY4&open=AaAUKVUhXMtocdt8EMY4&pullRequest=5882

Check failure on line 1979 in lib/PgSQL_Connection.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZY&open=AaAUKVYFXMtocdt8EMZY&pullRequest=5882

Check warning on line 64 in lib/PgSQL_Backend_Auth.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVUhXMtocdt8EMY7&open=AaAUKVUhXMtocdt8EMY7&pullRequest=5882

Check failure on line 2590 in lib/PgSQL_Connection.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZr&open=AaAUKVYFXMtocdt8EMZr&pullRequest=5882

Check failure on line 219 in lib/PgSQL_Connection.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVYFXMtocdt8EMZN&open=AaAUKVYFXMtocdt8EMZN&pullRequest=5882

Check failure on line 102 in lib/PgSQL_PreparedStatement.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rewrite the code so that you no longer need this "delete". 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVbeXMtocdt8EMaK&open=AaAUKVbeXMtocdt8EMaK&pullRequest=5882

Check failure on line 442 in test/tap/tests/unit/pgsql_backend_auth-t.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "free". 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVCPXMtocdt8EMYO&open=AaAUKVCPXMtocdt8EMYO&pullRequest=5882

Check failure on line 72 in lib/PgSQL_Backend_Auth.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure this weak hash algorithm is not used in a sensitive context here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVUhXMtocdt8EMY9&open=AaAUKVUhXMtocdt8EMY9&pullRequest=5882

Check failure on line 7018 in lib/PgSQL_Session.cpp

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=sysown_proxysql&issues=AaAUKVWvXMtocdt8EMZK&open=AaAUKVWvXMtocdt8EMZK&pullRequest=5882

Check warning on line 39 in test/tap/tests/unit/pgsql_stmt_meta_cache-t.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVMQXMtocdt8EMYW&open=AaAUKVMQXMtocdt8EMYW&pullRequest=5882

Check failure on line 3929 in lib/PgSQL_Session.cpp

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=sysown_proxysql&issues=AaAUKVWvXMtocdt8EMZJ&open=AaAUKVWvXMtocdt8EMZJ&pullRequest=5882

Check warning on line 131 in lib/PgSQL_Backend_Protocol.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here. 

See more on https://sonarcloud.io/project/issues?id=sysown_proxysql&issues=AaAUKVa0XMtocdt8EMaI&open=AaAUKVa0XMtocdt8EMaI&pullRequest=5882
View more details on SonarQubeCloud