Skip to content

Implement RBAC message discarding filters - #407

Open
magnusg-kdab wants to merge 2 commits into
mainfrom
rbacfilters
Open

Implement RBAC message discarding filters#407
magnusg-kdab wants to merge 2 commits into
mainfrom
rbacfilters

Add RBAC message discarding filters

73a842c
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Aug 13, 2026 in 14m 39s

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
36.5% Duplication on New Code (required ≤ 3%)
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 67 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused lambda capture "this". 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5F&open=AZ_7Mlpp3RRgPo7BMX5F&pullRequest=407

Check warning on line 102 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused parameter "rb", make it unnamed, or declare it "[[maybe_unused]]". 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5M&open=AZ_7Mlpp3RRgPo7BMX5M&pullRequest=407

Check warning on line 158 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Declaration shadows a local variable "reply" in the outer scope. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5P&open=AZ_7Mlpp3RRgPo7BMX5P&pullRequest=407

Check warning on line 67 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused parameter "replyContext", make it unnamed, or declare it "[[maybe_unused]]". 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5J&open=AZ_7Mlpp3RRgPo7BMX5J&pullRequest=407

Check warning on line 107 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Declaration shadows a local variable "reply" in the outer scope. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5K&open=AZ_7Mlpp3RRgPo7BMX5K&pullRequest=407

Check warning on line 67 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused parameter "context", make it unnamed, or declare it "[[maybe_unused]]". 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5H&open=AZ_7Mlpp3RRgPo7BMX5H&pullRequest=407

Check failure on line 99 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly capture the required scope variables. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5L&open=AZ_7Mlpp3RRgPo7BMX5L&pullRequest=407

Check failure on line 855 in src/majordomo/include/majordomo/Broker.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlq_3RRgPo7BMX5T&open=AZ_7Mlq_3RRgPo7BMX5T&pullRequest=407

Check warning on line 67 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused parameter "in", make it unnamed, or declare it "[[maybe_unused]]". 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5I&open=AZ_7Mlpp3RRgPo7BMX5I&pullRequest=407

Check failure on line 141 in .github/workflows/build_cmake.yml

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Avoid executing downloaded artifacts directly without verification. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7MltU3RRgPo7BMX5V&open=AZ_7MltU3RRgPo7BMX5V&pullRequest=407

Check warning on line 438 in src/client/include/Client.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "std::byte" for byte-oriented memory access. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlo63RRgPo7BMX5E&open=AZ_7Mlo63RRgPo7BMX5E&pullRequest=407

Check warning on line 67 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the unused parameter "rawCtx", make it unnamed, or declare it "[[maybe_unused]]". 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5G&open=AZ_7Mlpp3RRgPo7BMX5G&pullRequest=407

Check failure on line 151 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly capture the required scope variables. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5Q&open=AZ_7Mlpp3RRgPo7BMX5Q&pullRequest=407

Check failure on line 172 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly capture the required scope variables. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5S&open=AZ_7Mlpp3RRgPo7BMX5S&pullRequest=407

Check failure on line 158 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly capture the required scope variables. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5R&open=AZ_7Mlpp3RRgPo7BMX5R&pullRequest=407

Check warning on line 59 in src/services/include/services/KeyStore.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this const reference to "std::string" by a "std::string_view". 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlrc3RRgPo7BMX5U&open=AZ_7Mlrc3RRgPo7BMX5U&pullRequest=407

Check failure on line 107 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly capture the required scope variables. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5N&open=AZ_7Mlpp3RRgPo7BMX5N&pullRequest=407

Check failure on line 121 in src/majordomo/test/rbac_tests.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly capture the required scope variables. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlpp3RRgPo7BMX5O&open=AZ_7Mlpp3RRgPo7BMX5O&pullRequest=407

Check warning on line 435 in src/client/include/Client.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define and throw a dedicated exception instead of using a generic one. 

See more on https://sonarcloud.io/project/issues?id=fair-acc_opencmw-cpp&issues=AZ_7Mlo63RRgPo7BMX5D&open=AZ_7Mlo63RRgPo7BMX5D&pullRequest=407
View more details on SonarQubeCloud