Skip to content

DNS Provider URL Validation - #13821

Draft
DaanHoogland wants to merge 11 commits into
mainfrom
dnsProviderUrlValidate
Draft

DNS Provider URL Validation#13821
DaanHoogland wants to merge 11 commits into
mainfrom
dnsProviderUrlValidate

address review comments

738bc24
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 21, 2026 in 4m 9s

Quality Gate passed

Issues
9 New issues
0 Accepted issues

Measures
0 Security Hotspots
69.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 272 in server/src/main/java/org/apache/cloudstack/dns/DnsProviderManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.commons.lang3.StringUtils" for "isNotBlank". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAkP2qDObA3UIzKsBaa&open=AaAkP2qDObA3UIzKsBaa&pullRequest=13821

Check warning on line 892 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doReturn". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAUhlxVXt_QL6VJMwC4&open=AaAUhlxVXt_QL6VJMwC4&pullRequest=13821

Check warning on line 880 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doReturn". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AZ_b59q_GDdNQ584wbmu&open=AZ_b59q_GDdNQ584wbmu&pullRequest=13821

Check failure on line 273 in server/src/main/java/org/apache/cloudstack/dns/DnsProviderManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.commons.lang3.StringUtils" for "trim". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAVNSjxBuamWNefnd6r&open=AaAVNSjxBuamWNefnd6r&pullRequest=13821

Check failure on line 168 in server/src/main/java/org/apache/cloudstack/dns/DnsProviderManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.commons.lang3.StringUtils" for "isBlank". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAEkOQUYEZf7TajSZpo&open=AaAEkOQUYEZf7TajSZpo&pullRequest=13821

Check warning on line 893 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doReturn". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAUhlxVXt_QL6VJMwC5&open=AaAUhlxVXt_QL6VJMwC5&pullRequest=13821

Check warning on line 912 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a static import for "doReturn". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AZ_b59q_GDdNQ584wbmv&open=AZ_b59q_GDdNQ584wbmv&pullRequest=13821

Check failure on line 184 in server/src/main/java/org/apache/cloudstack/dns/DnsProviderManagerImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.commons.lang3.StringUtils" for "trim". 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAVNSjxBuamWNefnd6q&open=AaAVNSjxBuamWNefnd6q&pullRequest=13821

Check warning on line 850 in server/src/test/java/org/apache/cloudstack/dns/DnsProviderManagerImplTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=apache_cloudstack&issues=AaAkP2v8ObA3UIzKsBab&open=AaAkP2v8ObA3UIzKsBab&pullRequest=13821
View more details on SonarQubeCloud