Skip to content

Commit

Permalink
Polish
Browse files Browse the repository at this point in the history
  • Loading branch information
eddumelendez committed Sep 19, 2023
1 parent 6ab39e6 commit 6863ced
Show file tree
Hide file tree
Showing 4 changed files with 42 additions and 40 deletions.
Original file line number Diff line number Diff line change
@@ -1,12 +1,17 @@
package io.sdkman.automigration.config;

import io.sdkman.automigration.infrastructure.NoRedirectSimpleClientHttpRequestFactory;
import io.sdkman.automigration.infrastructure.ServerResponseErrorHandler;
import org.springframework.boot.web.client.RestTemplateBuilder;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
import org.springframework.http.HttpStatus;
import org.springframework.http.HttpStatusCode;
import org.springframework.http.client.SimpleClientHttpRequestFactory;
import org.springframework.web.client.DefaultResponseErrorHandler;
import org.springframework.web.client.RestTemplate;

import java.io.IOException;
import java.net.HttpURLConnection;

@Configuration
public class RestTemplateConfig {

Expand All @@ -17,4 +22,29 @@ public RestTemplate restTemplate(RestTemplateBuilder restTemplateBuilder) {
.build();
}

static class NoRedirectSimpleClientHttpRequestFactory extends SimpleClientHttpRequestFactory {

@Override
protected void prepareConnection(HttpURLConnection connection, String httpMethod) throws IOException {
super.prepareConnection(connection, httpMethod);
connection.setInstanceFollowRedirects(false);
}

}

static class ServerResponseErrorHandler extends DefaultResponseErrorHandler {

@Override
protected boolean hasError(int unknownStatusCode) {
HttpStatus.Series series = HttpStatus.Series.resolve(unknownStatusCode);
return series == HttpStatus.Series.SERVER_ERROR;
}

@Override
protected boolean hasError(HttpStatusCode statusCode) {
return statusCode.is5xxServerError();
}

}

}

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,5 +1,13 @@
package io.sdkman.automigration.models;

public record FoojayQueryParams(String distribution, String version, String javaVersion, String releaseStatus,
String os, boolean javafxBundled, String features, VendorProperties.OS vendorOsProperties) {
// @formatter:off
public record FoojayQueryParams(String distribution,
String version,
String javaVersion,
String releaseStatus,
String os,
boolean javafxBundled,
String features,
VendorProperties.OS vendorOsProperties) {
}
// @formatter:on

0 comments on commit 6863ced

Please sign in to comment.