Spring Data Rest:更改OpenAPI规范中的操作ID

Question

我正在尝试生成Spring Data Rest服务的openapi.yaml，这样我们就可以使用typescript angular生成器轻松生成客户端代码。不幸的是，生成的服务和方法的名称是。。。不太理想。我们为实体获得不同的控制器；搜索"；另一个是关系。此外，生成的服务中的函数名称非常长，没有添加太多信息/好处。这里有一个例子：

paths:
/pricingPlans:
get:
tags:
- pricing-plan-entity-controller
description: get-pricingplan
operationId: getCollectionResource-pricingplan-get_1

通过这个openapi.yaml，我们得到了一个带有函数getCollectionResource-pricingplan-get_1的PricingPlanEntityControllerService，这太荒谬了。我们想将其更改为PricingPlanService和getAll。

@Tag(name = "pricing-plan")
@CrossOrigin
public interface PricingPlanRepo extends CrudRepository<PricingPlan, UUID> {
@Override
Iterable<PricingPlan> findAll();

通过在类级别添加@Tag(name = "pricing-plan")，我们可以将生成的服务的名称更改为PricingPlanService，但无论我们尝试了什么，operationId始终保持原样。

我希望@Operation(operationId = "getAll")能做我们想做的事，但正如我所说：被忽视了。使用Spring Data Rest将所有这些注释应用于的正确方法是什么？

Answer 1

请注意，使用@Operation注释自定义操作id的方法不适用于大多数spring数据rest存储库：原因是操作是由框架内部生成的，您没有任何方法添加注释。

在所有情况下都有效的简单方法是使用OpenApiCustomiser来更改生成的OpenAPI规范的任何部分的值，如文档中所述。

@Bean
OpenApiCustomiser operationIdCustomiser() {
return openApi -> openApi.getPaths().values().stream().flatMap(pathItem -> pathItem.readOperations().stream())
.forEach(operation -> {
if ("id-to-change".equals(operation.getOperationId()))
operation.setOperationId("any id you want ...");
});
}

Answer 2

@mimi78向我介绍了如何自定义生成的OpenAPI规范。谢谢你！我关心的是简单地添加操作ID的1:1翻译的方法，因为内部/原始名称可能会随着端点的添加或删除而更改。我提出了一个解决方案，可以根据路径模式(例如/products/{id}/vendor(和HTTP方法生成操作ID。我认为这应该提供稳定的命名，这是人类可读的，更适合于基于操作ID的代码的客户端生成器。

我想分享这个解决方案，以防有一天其他人需要它：

@Configuration
public class OperationIdCustomizer {
@Bean
public OpenApiCustomiser operationIdCustomiser() {
// @formatter:off
return openApi -> openApi.getPaths().entrySet().stream()
.forEach(entry -> {
String path = entry.getKey();
PathItem pathItem = entry.getValue();
if (pathItem.getGet() != null)
pathItem.getGet().setOperationId(OperationIdGenerator.convert("get", path));
if (pathItem.getPost() != null)
pathItem.getPost().setOperationId(OperationIdGenerator.convert("post", path));
if (pathItem.getPut() != null)
pathItem.getPut().setOperationId(OperationIdGenerator.convert("put", path));
if (pathItem.getPatch() != null)
pathItem.getPatch().setOperationId(OperationIdGenerator.convert("patch", path));
if (pathItem.getDelete() != null)
pathItem.getDelete().setOperationId(OperationIdGenerator.convert("delete", path));
});
// @formatter:on
}
}

public class OperationIdGenerator {
private static String pattern1 = "^/([a-zA-Z]+)$"; // /products
private static String pattern2 = "^/([a-zA-Z]+)/(\{[a-zA-Z]+\})$"; // /products/{id}
private static String pattern3 = "^/([a-zA-Z]+)/(\{[a-zA-Z]+\})/([a-zA-Z]+)$"; // /products/{id}/vendor
private static String pattern4 = "^/([a-zA-Z]+)/(\{[a-zA-Z]+\})/([a-zA-Z]+)/(\{[a-zA-Z]+\})$"; // /products/{id}/vendor/{propertyId}
private static String pattern5 = "^/([a-zA-Z]+)/search/([a-zA-Z]+)$"; // /products/search/findByVendor
// @formatter:off
private static Map<String, String> httpMethodVerb = Map.of(
"get",      "get",
"post",     "create",
"put",      "replace",
"patch",    "update",
"delete",   "delete");
// @formatter:on
private static String handlePattern1(String op, String path) {
Pattern r = Pattern.compile(pattern1);
Matcher m = r.matcher(path);
boolean found = m.find();
if (!found)
return null;
String noun = toCamelCase(m.group(1));
String verb = getVerb(op);
if (verb.equals("create"))
noun = singularize(noun);
return verb + noun;
}
private static String handlePattern2(String op, String path) {
Pattern r = Pattern.compile(pattern2);
Matcher m = r.matcher(path);
boolean found = m.find();
if (!found)
return null;
String noun = toCamelCase(singularize(m.group(1)));
return getVerb(op) + noun;
}
private static String handlePattern3(String op, String path) {
Pattern r = Pattern.compile(pattern3);
Matcher m = r.matcher(path);
boolean found = m.find();
if (!found)
return null;
String noun = toCamelCase(singularize(m.group(1)));
String relation = toCamelCase(m.group(3));
return op + noun + relation;
}
private static String handlePattern4(String op, String path) {
Pattern r = Pattern.compile(pattern4);
Matcher m = r.matcher(path);
boolean found = m.find();
if (!found)
return null;

String entity = toCamelCase(singularize(m.group(1)));
String relation = m.group(3);
return getVerb(op) + entity + toCamelCase(singularize(relation)) + "ById";
}
private static String handlePattern5(String op, String path) {
Pattern r = Pattern.compile(pattern5);
Matcher m = r.matcher(path);
boolean found = m.find();
if (!found)
return null;
String entity = toCamelCase(m.group(1));
String searchMethod = m.group(2);
r = Pattern.compile("findBy([a-zA-Z0-9]+)");
m = r.matcher(searchMethod);
if (m.find())
return "search" + entity + "By" + toCamelCase(m.group(1));
return "search" + entity + "By" + toCamelCase(searchMethod);
}
public static String singularize(String word) {
Inflector i = new Inflector();
return i.singularize(word);
}
public static boolean isSingular(String word) {
Inflector i = new Inflector();
return i.singularize(word).equals(word);
}
public static String getVerb(String op) {
return httpMethodVerb.get(op);
}
public static String convert(String op, String path) {
String result = handlePattern1(op, path);
if (result == null) {
result = handlePattern2(op, path);
if (result == null) {
result = handlePattern3(op, path);
if (result == null) {
result = handlePattern4(op, path);
if (result == null) {
result = handlePattern5(op, path);
}
}
}
}
return result;
}
private static String toCamelCase(String phrase) {
List<String> words = new ArrayList<>();
for (String word : phrase.split("_"))
words.add(word);
for (int i = 0; i < words.size(); i++) {
String word = words.get(i);
String firstLetter = word.substring(0, 1).toUpperCase();
word = firstLetter + word.substring(1);
words.set(i, word);
}
return String.join("", words.toArray(new String[words.size()]));
}
}