Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .mvn/wrapper/maven-wrapper.properties
Original file line number Diff line number Diff line change
Expand Up @@ -14,5 +14,5 @@
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
distributionUrl=https://maven-central.storage-download.googleapis.com/maven2/org/apache/maven/apache-maven/3.8.6/apache-maven-3.8.6-bin.zip
wrapperUrl=https://maven-central.storage-download.googleapis.com/maven2/org/apache/maven/wrapper/maven-wrapper/3.1.1/maven-wrapper-3.1.1.jar
distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.9/apache-maven-3.9.9-bin.zip
wrapperUrl=https://repo.maven.apache.org/maven2/org/apache/maven/wrapper/maven-wrapper/3.3.2/maven-wrapper-3.3.2.jar
Empty file modified mvnw
100644 → 100755
Empty file.
16 changes: 12 additions & 4 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,14 @@ along with ShieldBlaze ExpressGateway. If not, see <https://www.gnu.org/license
<version>${netty}</version>
</dependency>

<!-- https://mvnrepository.com/artifact/io.netty/netty-transport-native-epoll -->
<dependency>
<groupId>io.netty</groupId>
<artifactId>netty-transport-native-epoll</artifactId>
<version>${netty}</version>
<classifier>linux-x86_64</classifier>
</dependency>

<!-- https://mvnrepository.com/artifact/io.netty/netty-tcnative -->
<dependency>
<groupId>io.netty</groupId>
Expand Down Expand Up @@ -262,10 +270,10 @@ along with ShieldBlaze ExpressGateway. If not, see <https://www.gnu.org/license
<artifactId>lightsail</artifactId>
</dependency>

<!-- https://mvnrepository.com/artifact/com.squareup.okhttp3/okhttp -->
<!-- https://mvnrepository.com/artifact/com.squareup.okhttp3/okhttp-jvm -->
<dependency>
<groupId>com.squareup.okhttp3</groupId>
<artifactId>okhttp</artifactId>
<artifactId>okhttp-jvm</artifactId>
<version>5.3.2</version>
</dependency>

Expand Down Expand Up @@ -404,8 +412,8 @@ along with ShieldBlaze ExpressGateway. If not, see <https://www.gnu.org/license
<artifactId>maven-compiler-plugin</artifactId>
<version>3.15.0</version>
<configuration>
<source>17</source>
<target>17</target>
<source>21</source>
<target>21</target>
</configuration>
</plugin>
<plugin>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ public final class ServiceDiscoveryClient {
false,
serviceDiscovery.hostnameVerification(),
serviceDiscovery.hostnameVerification(),
false,
false
)};

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ public ResponseEntity<String> deregister(@RequestBody Node node) throws Exceptio
}

@GetMapping(value = "get", produces = MediaType.APPLICATION_JSON_VALUE)
public ResponseEntity<String> get(@RequestParam String id) throws Exception {
public ResponseEntity<String> get(@RequestParam("id") String id) throws Exception {
ServiceInstance<Node> serviceInstance = serviceDiscovery.queryForInstance(SERVICE_NAME, id);

ObjectNode objectNode = OBJECT_MAPPER.createObjectNode();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@
import io.netty.handler.ssl.SslProvider;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import org.springframework.boot.web.embedded.netty.NettyReactiveWebServerFactory;
import org.springframework.boot.web.embedded.netty.NettyServerCustomizer;
import org.springframework.boot.reactor.netty.NettyReactiveWebServerFactory;
import org.springframework.boot.reactor.netty.NettyServerCustomizer;
import org.springframework.boot.web.server.WebServerFactoryCustomizer;
import org.springframework.context.annotation.Configuration;
import reactor.netty.http.Http2SslContextSpec;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,15 +28,21 @@
import org.junit.jupiter.api.TestMethodOrder;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.test.context.SpringBootTest;
import org.springframework.boot.test.web.client.TestRestTemplate;
import org.springframework.boot.test.web.server.LocalServerPort;
import org.springframework.http.HttpMethod;
import org.springframework.http.RequestEntity;
import org.springframework.http.ResponseEntity;
import org.springframework.http.client.SimpleClientHttpRequestFactory;
import org.springframework.web.client.RestTemplate;

import javax.net.ssl.HttpsURLConnection;
import javax.net.ssl.SSLContext;
import javax.net.ssl.TrustManager;
import javax.net.ssl.X509TrustManager;
import java.io.File;
import java.io.IOException;
import java.net.URI;
import java.security.cert.X509Certificate;

import static org.assertj.core.api.Assertions.assertThat;

Expand All @@ -55,7 +61,7 @@ class ServiceDiscoveryServerHttpsTest {
System.setProperty("config.file", absolutePath);
}

private final TestRestTemplate restTemplate = new TestRestTemplate(TestRestTemplate.HttpClientOption.SSL);
private final RestTemplate restTemplate = createTrustAllRestTemplate();

@LocalServerPort
private int ServerPort;
Expand All @@ -76,6 +82,25 @@ static void shutdown() throws IOException {
}
}

private static RestTemplate createTrustAllRestTemplate() {
try {
TrustManager[] trustAllCerts = new TrustManager[]{
new X509TrustManager() {
public X509Certificate[] getAcceptedIssuers() { return null; }
public void checkClientTrusted(X509Certificate[] certs, String authType) { }
public void checkServerTrusted(X509Certificate[] certs, String authType) { }
}
};
SSLContext sslContext = SSLContext.getInstance("TLS");
sslContext.init(null, trustAllCerts, new java.security.SecureRandom());
HttpsURLConnection.setDefaultSSLSocketFactory(sslContext.getSocketFactory());
HttpsURLConnection.setDefaultHostnameVerifier((hostname, session) -> true);
return new RestTemplate(new SimpleClientHttpRequestFactory());
} catch (Exception e) {
throw new RuntimeException(e);
}
}

@Order(1)
@Test
public void loadHandlerTest() {
Expand All @@ -89,7 +114,7 @@ public void registerServiceValidateSuccessful() {

ResponseEntity<String> response = restTemplate.exchange(request, String.class);
assertThat(response.getBody()).isNotNull();
assertThat(response.getStatusCodeValue()).isEqualTo(200);
assertThat(response.getStatusCode().value()).isEqualTo(200);
}

@Order(3)
Expand Down Expand Up @@ -117,6 +142,6 @@ public void deregisterServiceAndValidateSuccessful() {

ResponseEntity<String> response = restTemplate.exchange(request, String.class);
assertThat(response.getBody()).isNotNull();
assertThat(response.getStatusCodeValue()).isEqualTo(200);
assertThat(response.getStatusCode().value()).isEqualTo(200);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -28,11 +28,11 @@
import org.junit.jupiter.api.TestMethodOrder;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.test.context.SpringBootTest;
import org.springframework.boot.test.web.client.TestRestTemplate;
import org.springframework.boot.test.web.server.LocalServerPort;
import org.springframework.http.HttpMethod;
import org.springframework.http.RequestEntity;
import org.springframework.http.ResponseEntity;
import org.springframework.web.client.RestTemplate;

import java.io.File;
import java.io.IOException;
Expand All @@ -55,7 +55,7 @@ class ServiceDiscoveryServerTest {
System.setProperty("config.file", absolutePath);
}

private final TestRestTemplate restTemplate = new TestRestTemplate();
private final RestTemplate restTemplate = new RestTemplate();

@LocalServerPort
private int ServerPort;
Expand Down Expand Up @@ -89,7 +89,7 @@ public void registerServiceValidateSuccessful() {

ResponseEntity<String> response = restTemplate.exchange(request, String.class);
assertThat(response.getBody()).isNotNull();
assertThat(response.getStatusCodeValue()).isEqualTo(200);
assertThat(response.getStatusCode().value()).isEqualTo(200);
}

@Order(3)
Expand Down Expand Up @@ -117,6 +117,6 @@ public void deregisterServiceAndValidateSuccessful() {

ResponseEntity<String> response = restTemplate.exchange(request, String.class);
assertThat(response.getBody()).isNotNull();
assertThat(response.getStatusCodeValue()).isEqualTo(200);
assertThat(response.getStatusCode().value()).isEqualTo(200);
}
}
Loading