diff --git a/modules/swagger-codegen-cli/pom.xml b/modules/swagger-codegen-cli/pom.xml index 77271474dd5..a476851fb44 100644 --- a/modules/swagger-codegen-cli/pom.xml +++ b/modules/swagger-codegen-cli/pom.xml @@ -84,15 +84,9 @@ swagger-codegen-generators ${swagger-codegen-generators-version} - - io.swagger.core.v3 - swagger-core - ${swagger-core-version} - io.swagger.parser.v3 swagger-parser-v3 - ${swagger-parser-version} net.sourceforge.argparse4j diff --git a/modules/swagger-codegen/pom.xml b/modules/swagger-codegen/pom.xml index d244d636a0a..7fb3a8f5ffb 100644 --- a/modules/swagger-codegen/pom.xml +++ b/modules/swagger-codegen/pom.xml @@ -203,35 +203,13 @@ 2.4.2-SNAPSHOT - - io.swagger.core.v3 - swagger-models - ${swagger-core-version} - io.swagger swagger-parser - ${swagger-parser-version-v1} - - - io.swagger.core.v3 - swagger-core - ${swagger-core-version} - - - io.swagger.parser.v3 - swagger-parser-core - ${swagger-parser-version} - - - io.swagger.parser.v3 - swagger-parser-v3 - ${swagger-parser-version} io.swagger.parser.v3 swagger-parser - ${swagger-parser-version} io.swagger diff --git a/modules/swagger-codegen/src/main/java/io/swagger/codegen/v3/config/CodegenConfigurator.java b/modules/swagger-codegen/src/main/java/io/swagger/codegen/v3/config/CodegenConfigurator.java index f100756d977..515646caa29 100644 --- a/modules/swagger-codegen/src/main/java/io/swagger/codegen/v3/config/CodegenConfigurator.java +++ b/modules/swagger-codegen/src/main/java/io/swagger/codegen/v3/config/CodegenConfigurator.java @@ -437,7 +437,7 @@ public String loadSpecContent(String location, List auths) t data = ClasspathHelper.loadFileFromClasspath(location); } } - LOGGER.debug("Loaded raw data: {}", data); + LOGGER.trace("Loaded raw data: {}", data); return data; } @@ -470,6 +470,7 @@ public ClientOptInput toClientOptInput() { input.opts(new ClientOpts()) .openAPI(openAPI); + LOGGER.debug("getClientOptInput - parsed inputSpec"); } else { String specContent = null; try { @@ -492,7 +493,7 @@ public ClientOptInput toClientOptInput() { options.setFlatten(true); SwaggerParseResult result = new OpenAPIParser().readLocation(inputSpecURL, authorizationValues, options); OpenAPI openAPI = result.getOpenAPI(); - + LOGGER.debug("getClientOptInput - parsed inputSpecURL " + inputSpecURL); input.opts(new ClientOpts()) .openAPI(openAPI); } diff --git a/modules/swagger-codegen/src/main/java/io/swagger/codegen/v3/service/GeneratorUtil.java b/modules/swagger-codegen/src/main/java/io/swagger/codegen/v3/service/GeneratorUtil.java index 63f6ef7c25b..b021a215508 100644 --- a/modules/swagger-codegen/src/main/java/io/swagger/codegen/v3/service/GeneratorUtil.java +++ b/modules/swagger-codegen/src/main/java/io/swagger/codegen/v3/service/GeneratorUtil.java @@ -26,6 +26,7 @@ public class GeneratorUtil { protected static final Logger LOGGER = LoggerFactory.getLogger(GeneratorUtil.class); public static io.swagger.codegen.ClientOptInput getClientOptInputV2(GenerationRequest generationRequest) { + LOGGER.debug("getClientOptInputV2 - start"); final Options options = generationRequest.getOptions(); String inputSpec = null; if (generationRequest.getSpec() == null) { @@ -38,7 +39,7 @@ public static io.swagger.codegen.ClientOptInput getClientOptInputV2(GenerationRe String inputSpecURL = generationRequest.getSpecURL(); String lang = generationRequest.getLang(); validateSpec(lang, inputSpec, inputSpecURL); - + LOGGER.debug("getClientOptInputV2 - spec validated"); final List authorizationValues = io.swagger.codegen.auth.AuthParser.parse(generationRequest.getOptions().getAuth()); if (generationRequest.getOptions().getAuthorizationValue() != null) { io.swagger.models.auth.AuthorizationValue authorizationValue = new io.swagger.models.auth.AuthorizationValue() @@ -47,7 +48,7 @@ public static io.swagger.codegen.ClientOptInput getClientOptInputV2(GenerationRe .type(generationRequest.getOptions().getAuthorizationValue().getType()); authorizationValues.add(authorizationValue); } - + LOGGER.debug("getClientOptInputV2 - processed auth"); Swagger swagger; if (StringUtils.isBlank(inputSpec)) { @@ -62,6 +63,7 @@ public static io.swagger.codegen.ClientOptInput getClientOptInputV2(GenerationRe } else { throw new BadRequestException("No swagger specification was supplied"); } + LOGGER.debug("getClientOptInputV2 - parsed inputSpecURL " + inputSpecURL); } else { try { JsonNode node = io.swagger.util.Json.mapper().readTree(inputSpec); @@ -74,6 +76,7 @@ public static io.swagger.codegen.ClientOptInput getClientOptInputV2(GenerationRe LOGGER.error("Exception parsing input spec", e); throw new BadRequestException("The swagger specification supplied was not valid"); } + LOGGER.debug("getClientOptInputV2 - parsed inputSpec"); } if (swagger == null) { throw new BadRequestException("The swagger specification supplied was not valid"); @@ -160,6 +163,7 @@ public static io.swagger.codegen.ClientOptInput getClientOptInputV2(GenerationRe codegenConfig.additionalProperties().put("swagger", swagger); clientOptInput.setConfig(codegenConfig); + LOGGER.debug("getClientOptInputV2 - end"); return clientOptInput; } @@ -172,6 +176,7 @@ public static void validateSpec(String lang, String inputSpec, String inputSpecU } public static ClientOptInput getClientOptInput(GenerationRequest generationRequest) { + LOGGER.debug("getClientOptInput - start"); final Options options = generationRequest.getOptions(); String inputSpec = null; if (generationRequest.getSpec() == null) { @@ -184,7 +189,7 @@ public static ClientOptInput getClientOptInput(GenerationRequest generationReque String inputSpecURL = generationRequest.getSpecURL(); String lang = generationRequest.getLang(); validateSpec(lang, inputSpec, inputSpecURL); - + LOGGER.debug("getClientOptInput - validated"); CodegenConfigurator configurator = new CodegenConfigurator(); configurator.setOutputDir(generationRequest.getOptions().getOutputDir()); @@ -278,7 +283,7 @@ public static ClientOptInput getClientOptInput(GenerationRequest generationReque configurator.addAdditionalReservedWordMapping(entry.getKey(), entry.getValue()); } } - + LOGGER.debug("getClientOptInput - end"); return configurator.toClientOptInput(); } } diff --git a/modules/swagger-generator/pom.xml b/modules/swagger-generator/pom.xml index 845b96f60aa..eb3a58f4b35 100644 --- a/modules/swagger-generator/pom.xml +++ b/modules/swagger-generator/pom.xml @@ -289,11 +289,6 @@ - - io.swagger - swagger-parser - ${swagger-parser-version-v1} - javax.servlet javax.servlet-api @@ -307,7 +302,6 @@ io.swagger.codegen.v3 swagger-codegen-generators - ${swagger-codegen-generators-version} io.swagger @@ -330,7 +324,7 @@ true unstable 1.0.0 - 9.4.12.v20180830 + 9.4.14.v20181114 2.0.0-rc2 4.8.2 3.1.0 diff --git a/modules/swagger-generator/src/main/java/io/swagger/v3/generator/online/GeneratorController.java b/modules/swagger-generator/src/main/java/io/swagger/v3/generator/online/GeneratorController.java index 78585495e57..559254d4adc 100644 --- a/modules/swagger-generator/src/main/java/io/swagger/v3/generator/online/GeneratorController.java +++ b/modules/swagger-generator/src/main/java/io/swagger/v3/generator/online/GeneratorController.java @@ -322,7 +322,7 @@ public ResponseContext listOptions(RequestContext requestContext, String languag public ResponseContext generateFromURL(RequestContext context, String codegenOptionsURL) { final String content; - + LOGGER.debug("generateFromURL start - " + codegenOptionsURL); try { content = RemoteUrl.urlToString(codegenOptionsURL, null); } catch (Exception e) { @@ -359,11 +359,64 @@ public ResponseContext generateFromURL(RequestContext context, String codegenOpt } } - return generate(context, generationRequest); + ResponseContext responseContext = generate(context, generationRequest); + LOGGER.debug("generateFromURL end - " + codegenOptionsURL); + return responseContext; + + } + private String requestLog(GenerationRequest generationRequest) { + final int maxLength = 41; + StringBuffer requestLog = new StringBuffer(); + requestLog.append("lang: "); + requestLog.append(generationRequest.getLang()); + requestLog.append(", "); + requestLog.append("version: "); + requestLog.append(generationRequest.getCodegenVersion().name()); + requestLog.append(", "); + requestLog.append("specURL: "); + requestLog.append(generationRequest.getSpecURL()); + requestLog.append(", "); + if (generationRequest.getSpec() != null) { + String spec = null; + if (generationRequest.getSpec() instanceof String && StringUtils.isNotBlank((String) generationRequest.getSpec())) { + if (((String) generationRequest.getSpec()).length() > maxLength) { + spec = ((String) generationRequest.getSpec()).substring(0, 40); + } else { + spec = ((String) generationRequest.getSpec()); + } + requestLog.append("spec: "); + requestLog.append(spec); + requestLog.append(", "); + } else { + try { + spec = Json.pretty(generationRequest.getSpec()); + if (spec.length() > maxLength) { + spec = spec.substring(0, 40); + } + requestLog.append("spec: "); + requestLog.append(spec); + requestLog.append(", "); + } catch (Exception e) { + requestLog.append("spec: "); + requestLog.append(spec); + requestLog.append(", "); + } + } + } + if (generationRequest.getOptions() != null) { + if (StringUtils.isNotBlank(generationRequest.getOptions().getLibrary())) { + requestLog.append("lib: "); + requestLog.append(generationRequest.getOptions().getLibrary()); + } + } + return requestLog.toString(); } public ResponseContext generate(RequestContext context, GenerationRequest generationRequest) { + + String requestLog = requestLog(generationRequest); + LOGGER.debug("generate start - " + requestLog); File outputRootFolder = getTmpFolder(); String destPath = null; @@ -406,7 +459,9 @@ public ResponseContext generate(RequestContext context, GenerationRequest genera LOGGER.info("file zip file: " + outputFile.getAbsolutePath()); - return generate(generationRequest, outputRootFolder, outputContentFolder, outputFile); + ResponseContext responseContext = generate(generationRequest, outputRootFolder, outputContentFolder, outputFile); + LOGGER.debug("generate end - " + requestLog); + return responseContext; } diff --git a/modules/swagger-generator/src/main/resources/logback.xml b/modules/swagger-generator/src/main/resources/logback.xml index 82ac57f5209..e6270d9b01b 100644 --- a/modules/swagger-generator/src/main/resources/logback.xml +++ b/modules/swagger-generator/src/main/resources/logback.xml @@ -8,6 +8,12 @@ + + diff --git a/pom.docker.xml b/pom.docker.xml index 8d786f81f43..13141bd21ca 100644 --- a/pom.docker.xml +++ b/pom.docker.xml @@ -900,6 +900,127 @@ + + + io.swagger.core.v3 + swagger-core + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-annotations + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-models + ${swagger-core-version} + + + + io.swagger + swagger-core + ${swagger-core-version-v1} + + + io.swagger + swagger-annotations + ${swagger-core-version-v1} + + + io.swagge + swagger-models + ${swagger-core-version-v1} + + + + io.swagger.parser.v3 + swagger-parser + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-core + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-v3 + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-v2-converter + ${swagger-parser-version} + + + + io.swagger + swagger-parser + ${swagger-parser-version-v1} + + + + io.swagger.codegen.v3 + swagger-codegen-generators + ${swagger-codegen-generators-version} + + + + com.fasterxml.jackson.core + jackson-databind + ${jackson-version} + + + com.fasterxml.jackson.core + jackson-annotations + ${jackson-version} + + + com.fasterxml.jackson.core + jackson-core + ${jackson-version} + + + + com.fasterxml.jackson.module + jackson-module-jaxb-annotations + ${jackson-version} + + + + com.fasterxml.jackson.dataformat + jackson-dataformat-xml + ${jackson-version} + + + com.fasterxml.jackson.dataformat + jackson-dataformat-yaml + ${jackson-version} + + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-base + ${jackson-version} + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-json-provider + ${jackson-version} + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-xml-provider + ${jackson-version} + + + + com.fasterxml.jackson.datatype + jackson-datatype-joda + ${jackson-version} + + junit junit @@ -943,15 +1064,17 @@ + 1.0.5-SNAPSHOT + 2.0.7-SNAPSHOT + 1.5.22-SNAPSHOT 2.0.9-SNAPSHOT 1.0.42-SNAPSHOT + 2.9.8 2.11.1 3.3.0 - 2.0.6 2.4 1.2 4.8.1 - 2.8.9 1.0.0 3.4 1.7.25 @@ -959,9 +1082,9 @@ 3.2.1 1.12 6.9.6 - 2.19.1 + 2.22.0 1.25 - 0.9.10 - 1.0.5-SNAPSHOT + 0.9.11 + 1.42 diff --git a/pom.xml b/pom.xml index 6f3b4027cf4..117a993b69e 100644 --- a/pom.xml +++ b/pom.xml @@ -986,6 +986,132 @@ + + + io.swagger.core.v3 + swagger-jaxrs2 + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-core + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-annotations + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-models + ${swagger-core-version} + + + + io.swagger + swagger-core + ${swagger-core-version-v1} + + + io.swagger + swagger-annotations + ${swagger-core-version-v1} + + + io.swagge + swagger-models + ${swagger-core-version-v1} + + + + io.swagger.parser.v3 + swagger-parser + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-core + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-v3 + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-v2-converter + ${swagger-parser-version} + + + + io.swagger + swagger-parser + ${swagger-parser-version-v1} + + + + io.swagger.codegen.v3 + swagger-codegen-generators + ${swagger-codegen-generators-version} + + + + com.fasterxml.jackson.core + jackson-databind + ${jackson-version} + + + com.fasterxml.jackson.core + jackson-annotations + ${jackson-version} + + + com.fasterxml.jackson.core + jackson-core + ${jackson-version} + + + + com.fasterxml.jackson.module + jackson-module-jaxb-annotations + ${jackson-version} + + + + com.fasterxml.jackson.dataformat + jackson-dataformat-xml + ${jackson-version} + + + com.fasterxml.jackson.dataformat + jackson-dataformat-yaml + ${jackson-version} + + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-base + ${jackson-version} + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-json-provider + ${jackson-version} + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-xml-provider + ${jackson-version} + + + + com.fasterxml.jackson.datatype + jackson-datatype-joda + ${jackson-version} + + junit junit @@ -1029,15 +1155,17 @@ + 1.0.5-SNAPSHOT + 2.0.7-SNAPSHOT + 1.5.22-SNAPSHOT 2.0.9-SNAPSHOT 1.0.42-SNAPSHOT + 2.9.8 2.11.1 3.3.0 - 2.0.6 2.4 1.2 4.8.1 - 2.8.9 1.0.0 3.4 1.7.25 @@ -1047,8 +1175,7 @@ 6.9.6 2.22.0 1.25 - 0.9.10 - 1.0.5-SNAPSHOT + 0.9.11 1.42 diff --git a/pom.xml.bash b/pom.xml.bash index 64481a1b86d..37f6aafef25 100644 --- a/pom.xml.bash +++ b/pom.xml.bash @@ -888,6 +888,127 @@ + + + io.swagger.core.v3 + swagger-core + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-annotations + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-models + ${swagger-core-version} + + + + io.swagger + swagger-core + ${swagger-core-version-v1} + + + io.swagger + swagger-annotations + ${swagger-core-version-v1} + + + io.swagge + swagger-models + ${swagger-core-version-v1} + + + + io.swagger.parser.v3 + swagger-parser + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-core + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-v3 + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-v2-converter + ${swagger-parser-version} + + + + io.swagger + swagger-parser + ${swagger-parser-version-v1} + + + + io.swagger.codegen.v3 + swagger-codegen-generators + ${swagger-codegen-generators-version} + + + + com.fasterxml.jackson.core + jackson-databind + ${jackson-version} + + + com.fasterxml.jackson.core + jackson-annotations + ${jackson-version} + + + com.fasterxml.jackson.core + jackson-core + ${jackson-version} + + + + com.fasterxml.jackson.module + jackson-module-jaxb-annotations + ${jackson-version} + + + + com.fasterxml.jackson.dataformat + jackson-dataformat-xml + ${jackson-version} + + + com.fasterxml.jackson.dataformat + jackson-dataformat-yaml + ${jackson-version} + + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-base + ${jackson-version} + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-json-provider + ${jackson-version} + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-xml-provider + ${jackson-version} + + + + com.fasterxml.jackson.datatype + jackson-datatype-joda + ${jackson-version} + + junit junit @@ -931,15 +1052,17 @@ + 1.0.5-SNAPSHOT + 2.0.7-SNAPSHOT + 1.5.22-SNAPSHOT 2.0.9-SNAPSHOT 1.0.42-SNAPSHOT + 2.9.8 2.11.1 3.3.0 - 2.0.6 2.4 1.2 4.8.1 - 2.8.9 1.0.0 3.4 1.7.25 @@ -949,8 +1072,7 @@ 6.9.6 2.22.0 1.25 - 0.9.10 - 1.0.5-SNAPSHOT + 0.9.11 1.42 diff --git a/pom.xml.shippable b/pom.xml.shippable index 1783b4cd914..656f0486782 100644 --- a/pom.xml.shippable +++ b/pom.xml.shippable @@ -890,6 +890,127 @@ + + + io.swagger.core.v3 + swagger-core + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-annotations + ${swagger-core-version} + + + io.swagger.core.v3 + swagger-models + ${swagger-core-version} + + + + io.swagger + swagger-core + ${swagger-core-version-v1} + + + io.swagger + swagger-annotations + ${swagger-core-version-v1} + + + io.swagge + swagger-models + ${swagger-core-version-v1} + + + + io.swagger.parser.v3 + swagger-parser + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-core + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-v3 + ${swagger-parser-version} + + + io.swagger.parser.v3 + swagger-parser-v2-converter + ${swagger-parser-version} + + + + io.swagger + swagger-parser + ${swagger-parser-version-v1} + + + + io.swagger.codegen.v3 + swagger-codegen-generators + ${swagger-codegen-generators-version} + + + + com.fasterxml.jackson.core + jackson-databind + ${jackson-version} + + + com.fasterxml.jackson.core + jackson-annotations + ${jackson-version} + + + com.fasterxml.jackson.core + jackson-core + ${jackson-version} + + + + com.fasterxml.jackson.module + jackson-module-jaxb-annotations + ${jackson-version} + + + + com.fasterxml.jackson.dataformat + jackson-dataformat-xml + ${jackson-version} + + + com.fasterxml.jackson.dataformat + jackson-dataformat-yaml + ${jackson-version} + + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-base + ${jackson-version} + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-json-provider + ${jackson-version} + + + com.fasterxml.jackson.jaxrs + jackson-jaxrs-xml-provider + ${jackson-version} + + + + com.fasterxml.jackson.datatype + jackson-datatype-joda + ${jackson-version} + + junit junit @@ -933,15 +1054,17 @@ + 1.0.5-SNAPSHOT + 2.0.7-SNAPSHOT + 1.5.22-SNAPSHOT 2.0.9-SNAPSHOT 1.0.42-SNAPSHOT + 2.9.8 2.11.1 3.3.0 - 2.0.6 2.4 1.2 4.8.1 - 2.8.9 1.0.0 3.4 1.7.25 @@ -951,8 +1074,7 @@ 6.9.6 2.22.0 1.25 - 0.9.10 - 1.0.5-SNAPSHOT + 0.9.11 1.42