diff --git a/snail-job-client/snail-job-client-common/src/main/java/com/aizuda/snailjob/client/common/config/SnailJobProperties.java b/snail-job-client/snail-job-client-common/src/main/java/com/aizuda/snailjob/client/common/config/SnailJobProperties.java index 5f7bfe02d..6da5e17b5 100644 --- a/snail-job-client/snail-job-client-common/src/main/java/com/aizuda/snailjob/client/common/config/SnailJobProperties.java +++ b/snail-job-client/snail-job-client-common/src/main/java/com/aizuda/snailjob/client/common/config/SnailJobProperties.java @@ -70,6 +70,11 @@ public class SnailJobProperties { */ private ServerConfig server = new ServerConfig(); + /** + * 内置http执行器自定义响应结果配置 + */ + private HttpResponse httpResponse = new HttpResponse(); + /** * 重试模块配置 */ @@ -109,6 +114,24 @@ public class SnailJobProperties { private int port = 17888; } + @Data + public static class HttpResponse { + /** + * 内置http执行器响应成功状态码,默认值200 + */ + private int code; + + /** + * 内置http执行器状态码字段名称,默认值code,只针对responseType等于json生效 + */ + private String field; + + /** + * 内置http执行器响应类型,可选值json或者text,默认值json + */ + private String responseType; + } + @Data public static class SlidingWindowConfig { diff --git a/snail-job-client/snail-job-client-job-core/src/main/java/com/aizuda/snailjob/client/job/core/executor/builtin/AbstractHttpExecutor.java b/snail-job-client/snail-job-client-job-core/src/main/java/com/aizuda/snailjob/client/job/core/executor/builtin/AbstractHttpExecutor.java index 24e406d2a..fc5b4d9b7 100644 --- a/snail-job-client/snail-job-client-job-core/src/main/java/com/aizuda/snailjob/client/job/core/executor/builtin/AbstractHttpExecutor.java +++ b/snail-job-client/snail-job-client-job-core/src/main/java/com/aizuda/snailjob/client/job/core/executor/builtin/AbstractHttpExecutor.java @@ -14,6 +14,7 @@ import org.springframework.util.StringUtils; import java.util.Base64; import java.util.Map; import java.util.Objects; +import java.util.Optional; import java.util.regex.Pattern; @@ -55,14 +56,7 @@ public abstract class AbstractHttpExecutor { private ExecuteResult executeRequestAndHandleResponse(HttpRequest httpRequest) { try (HttpResponse response = httpRequest.execute()) { - int errCode = response.getStatus(); - String body = response.body(); - if (errCode != HTTP_SUCCESS_CODE) { - SnailJobLog.LOCAL.error("{} request to URL: {} failed with code: {}, response body: {}", - httpRequest.getMethod(), httpRequest.getUrl(), errCode, body); - return ExecuteResult.failure("HTTP request failed"); - } - return ExecuteResult.success(body); + return validateResponse(response, httpRequest, snailJobProperties.getHttpResponse()); } catch (Exception e) { throw new SnailJobInnerExecutorException("[snail-job] HTTP internal executor failed", e); } @@ -75,6 +69,93 @@ public abstract class AbstractHttpExecutor { httpParams.setUrl(httpParams.getUrl().startsWith(HTTP) ? httpParams.getUrl() : HTTP_PREFIX + httpParams.getUrl()); } + /** + * 验证http响应是否有效,并根据响应类型进行进一步验证 + * + * @param response + * @param httpRequest + * @param httpResponse + * @return + */ + private ExecuteResult validateResponse(HttpResponse response, HttpRequest httpRequest, SnailJobProperties.HttpResponse httpResponse) { + int errCode = response.getStatus(); + String body = response.body(); + // 检查http响应状态码是否为成功状态码 + if (errCode != HTTP_SUCCESS_CODE) { + SnailJobLog.LOCAL.error("{} request to URL: {} failed with code: {}, response body: {}", + httpRequest.getMethod(), httpRequest.getUrl(), errCode, body); + return ExecuteResult.failure("HTTP request failed"); + } + // 如果配置了httpResponse,则根据响应类型进行进一步验证 + if (Objects.nonNull(httpResponse)) { + int code = Optional.of(httpResponse.getCode()).orElse(HTTP_SUCCESS_CODE); + String field = Optional.of(httpResponse.getField()).orElse("code"); + String responseType = Optional.of(httpResponse.getResponseType()).orElse("json"); + // 根据不同的响应类型进行验证 + if ("json".equalsIgnoreCase(responseType)) { + return validateJsonResponse(body, code, field, httpRequest); + } else if ("text".equalsIgnoreCase(responseType)) { + return validateTextResponse(body, code, httpRequest); + } else { + return ExecuteResult.failure("the responseType is not json or text"); + } + } + return ExecuteResult.success(body); + } + + /** + * 验证json响应类型 + * + * @param body + * @param code + * @param field + * @param httpRequest + * @return + */ + private ExecuteResult validateJsonResponse(String body, int code, String field, HttpRequest httpRequest) { + // 检查响应体是否为json格式 + if (!JsonUtil.isValidJson(body) || JsonUtil.isEmptyJson(body)) { + SnailJobLog.LOCAL.error("the responseType is json,but the response body fails to validate json or json is empty"); + return ExecuteResult.failure("the responseType is json,but the response body fails to validate json or json is empty"); + } + // 检查响应体是否包含指定的状态码字段 + Map objectObjectMap = JsonUtil.parseHashMap(body); + if (!objectObjectMap.containsKey(field)) { + SnailJobLog.LOCAL.error("the responseType is json,but there is no status code field:" + field); + return ExecuteResult.failure("the responseType is json,but there is no status code field:" + field); + } + // 检查响应体中状态码是否与指定的状态码是否一致 + if (!Objects.equals(code, objectObjectMap.get(field))) { + SnailJobLog.LOCAL.error("{} request to URL: {} failed with code: {}, response body: {}", + httpRequest.getMethod(), httpRequest.getUrl(), code, body); + return ExecuteResult.failure("the response status code is not equal to the specified status code"); + } + return ExecuteResult.success(body); + } + + /** + * 验证text响应类型 + * + * @param body + * @param code + * @param httpRequest + * @return + */ + private ExecuteResult validateTextResponse(String body, int code, HttpRequest httpRequest) { + // 检查响应体是否为空 + if (!StringUtils.hasLength(body)) { + SnailJobLog.LOCAL.error("the responseType is text,but the response body is empty"); + return ExecuteResult.failure("the responseType is text,but the response body is empty"); + } + // 检查响应体是否与指定的状态码是否一致 + if (!Objects.equals(code + "", body)) { + SnailJobLog.LOCAL.error("{} request to URL: {} failed with code: {}, response body: {}", + httpRequest.getMethod(), httpRequest.getUrl(), code, body); + return ExecuteResult.failure("the response status code is not equal to the specified status code"); + } + return ExecuteResult.success(body); + } + private void setDefaultMethodAndBody(HttpParams httpParams) { if (StringUtils.isEmpty(httpParams.getMethod())) { httpParams.setMethod(DEFAULT_REQUEST_METHOD);