Skip to content
This repository was archived by the owner on Apr 10, 2024. It is now read-only.

improvement: use etcd health check #78

Merged
merged 3 commits into from
Apr 10, 2023
Merged

improvement: use etcd health check #78

merged 3 commits into from
Apr 10, 2023

Conversation

csviri
Copy link
Collaborator

@csviri csviri commented Apr 10, 2023

No description provided.

@csviri csviri self-assigned this Apr 10, 2023
github-advanced-security[bot]
github-advanced-security bot found potential problems Apr 10, 2023
View reviewed changes
core/src/main/java/io/javaoperatorsdk/jenvtest/process/ProcessReadinessChecker.java
final Socket a_socket) {}

@Override
public void checkClientTrusted(X509Certificate[] chain, String authType,

Check failure

Code scanning / SonarCloud

Server certificates should be verified during SSL/TLS connections

<!--SONAR_ISSUE_KEY:AYdrAKP6ZeBsqSlr6o8m-->Enable server certificate validation on this SSL/TLS connection. <p>See more on <a href="https://sonarcloud.io/project/issues?id=java-operator-sdk_jenvtest&issues=AYdrAKP6ZeBsqSlr6o8m&open=AYdrAKP6ZeBsqSlr6o8m&pullRequest=78">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/ProcessReadinessChecker.java

}

public void checkServerTrusted(

Check failure

Code scanning / SonarCloud

Server certificates should be verified during SSL/TLS connections

<!--SONAR_ISSUE_KEY:AYdrAKP6ZeBsqSlr6o8n-->Enable server certificate validation on this SSL/TLS connection. <p>See more on <a href="https://sonarcloud.io/project/issues?id=java-operator-sdk_jenvtest&issues=AYdrAKP6ZeBsqSlr6o8n&open=AYdrAKP6ZeBsqSlr6o8n&pullRequest=78">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/ProcessReadinessChecker.java
new TrustManager[] {
new X509ExtendedTrustManager() {
@Override
public void checkClientTrusted(X509Certificate[] chain, String authType,

Check failure

Code scanning / SonarCloud

Server certificates should be verified during SSL/TLS connections

<!--SONAR_ISSUE_KEY:AYdrAKP6ZeBsqSlr6o8h-->Enable server certificate validation on this SSL/TLS connection. <p>See more on <a href="https://sonarcloud.io/project/issues?id=java-operator-sdk_jenvtest&issues=AYdrAKP6ZeBsqSlr6o8h&open=AYdrAKP6ZeBsqSlr6o8h&pullRequest=78">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/ProcessReadinessChecker.java
return null;
}

public void checkClientTrusted(

Check failure

Code scanning / SonarCloud

Server certificates should be verified during SSL/TLS connections

<!--SONAR_ISSUE_KEY:AYdrAKP6ZeBsqSlr6o8j-->Enable server certificate validation on this SSL/TLS connection. <p>See more on <a href="https://sonarcloud.io/project/issues?id=java-operator-sdk_jenvtest&issues=AYdrAKP6ZeBsqSlr6o8j&open=AYdrAKP6ZeBsqSlr6o8j&pullRequest=78">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/ProcessReadinessChecker.java
final X509Certificate[] a_certificates,
final String a_auth_type) {}

public void checkServerTrusted(

Check failure

Code scanning / SonarCloud

Server certificates should be verified during SSL/TLS connections

<!--SONAR_ISSUE_KEY:AYdrAKP6ZeBsqSlr6o8k-->Enable server certificate validation on this SSL/TLS connection. <p>See more on <a href="https://sonarcloud.io/project/issues?id=java-operator-sdk_jenvtest&issues=AYdrAKP6ZeBsqSlr6o8k&open=AYdrAKP6ZeBsqSlr6o8k&pullRequest=78">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/ProcessReadinessChecker.java
final String a_auth_type) {}


public void checkServerTrusted(

Check failure

Code scanning / SonarCloud

Server certificates should be verified during SSL/TLS connections

<!--SONAR_ISSUE_KEY:AYdrAKP6ZeBsqSlr6o8l-->Enable server certificate validation on this SSL/TLS connection. <p>See more on <a href="https://sonarcloud.io/project/issues?id=java-operator-sdk_jenvtest&issues=AYdrAKP6ZeBsqSlr6o8l&open=AYdrAKP6ZeBsqSlr6o8l&pullRequest=78">SonarCloud</a></p>
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability D 6 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 16 Code Smells

73.9% 73.9% Coverage
0.0% 0.0% Duplication

@csviri csviri merged commit f632843 into main Apr 10, 2023
@csviri csviri deleted the etcd-health-endpoint branch April 10, 2023 13:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@csviri csviri

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@csviri