SpringCloud/spring-cloud-openfeign
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Bumping versions

Browse Source
This commit is contained in:
spring-builds
2025-05-23 23:45:54 +00:00
parent deabba6227
commit e60d432689
1 changed files with 7 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
spring-cloud-openfeign-core/src/main/java/org/springframework/cloud/openfeign/support/SpringMvcContract.java
View File

@@ -241,12 +241,10 @@ public class SpringMvcContract extends Contract.BaseContract implements Resource
if (isGetMethod(metadata) && method.getParameterCount() > 0 && !hasHttpAnnotations(method)) {
if (LOG.isWarnEnabled()) {
LOG.warn(String.format(
"[OpenFeign Warning] Feign method '%s' is declared as GET with parameters, but none of the parameters are annotated " +
"(e.g. @RequestParam, @RequestHeader, @PathVariable, etc). This may result in fallback to POST at runtime. " +
"Consider explicitly annotating parameters.",
method.toGenericString()
));
LOG.warn(String
.format("[OpenFeign Warning] Feign method '%s' is declared as GET with parameters, but none of the parameters are annotated "
+ "(e.g. @RequestParam, @RequestHeader, @PathVariable, etc). This may result in fallback to POST at runtime. "
+ "Consider explicitly annotating parameters.", method.toGenericString()));
}
}
@@ -261,11 +259,9 @@ public class SpringMvcContract extends Contract.BaseContract implements Resource
for (Parameter parameter : method.getParameters()) {
for (Annotation annotation : parameter.getAnnotations()) {
Class<? extends Annotation> annotationType = annotation.annotationType();
if (annotationType == RequestParam.class ||
annotationType == RequestHeader.class ||
annotationType == PathVariable.class ||
annotationType == SpringQueryMap.class ||
annotationType == QueryMap.class) {
if (annotationType == RequestParam.class || annotationType == RequestHeader.class
|| annotationType == PathVariable.class || annotationType == SpringQueryMap.class
|| annotationType == QueryMap.class) {
return true;
}
}