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

feat: readyz polling for startup #56

Merged
merged 7 commits into from
Mar 29, 2023
Merged

feat: readyz polling for startup #56

merged 7 commits into from
Mar 29, 2023

Conversation

csviri
Copy link
Collaborator

@csviri csviri commented Mar 29, 2023

No description provided.

@csviri csviri self-assigned this Mar 29, 2023
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 29, 2023
View reviewed changes
core/src/main/java/io/javaoperatorsdk/jenvtest/process/KubeAPIServerProcess.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:AYcs9LV_zKUzN5JdWgI0-->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=AYcs9LV_zKUzN5JdWgI0&open=AYcs9LV_zKUzN5JdWgI0&pullRequest=56">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/KubeAPIServerProcess.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:AYcs9LV_zKUzN5JdWgI1-->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=AYcs9LV_zKUzN5JdWgI1&open=AYcs9LV_zKUzN5JdWgI1&pullRequest=56">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/KubeAPIServerProcess.java

}

public void checkServerTrusted(

Check failure

Code scanning / SonarCloud

Server certificates should be verified during SSL/TLS connections

<!--SONAR_ISSUE_KEY:AYcs9LV_zKUzN5JdWgI2-->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=AYcs9LV_zKUzN5JdWgI2&open=AYcs9LV_zKUzN5JdWgI2&pullRequest=56">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/KubeAPIServerProcess.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:AYcs9LV_zKUzN5JdWgIw-->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=AYcs9LV_zKUzN5JdWgIw&open=AYcs9LV_zKUzN5JdWgIw&pullRequest=56">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/KubeAPIServerProcess.java
return null;
}

public void checkClientTrusted(

Check failure

Code scanning / SonarCloud

Server certificates should be verified during SSL/TLS connections

<!--SONAR_ISSUE_KEY:AYcs9LV_zKUzN5JdWgIy-->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=AYcs9LV_zKUzN5JdWgIy&open=AYcs9LV_zKUzN5JdWgIy&pullRequest=56">SonarCloud</a></p>
core/src/main/java/io/javaoperatorsdk/jenvtest/process/KubeAPIServerProcess.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:AYcs9LV_zKUzN5JdWgIz-->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=AYcs9LV_zKUzN5JdWgIz&open=AYcs9LV_zKUzN5JdWgIz&pullRequest=56">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 15 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@csviri csviri merged commit f72b7a1 into main Mar 29, 2023
@csviri csviri deleted the health-polling-startup branch March 29, 2023 11:12
@csviri csviri mentioned this pull request Mar 29, 2023
Closed
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