Skip to content

Commit 8c40d4e

Browse files
committed
fix: do not return violations that should be ignored
1 parent f3b3372 commit 8c40d4e

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

openapi-validation-core/src/main/java/com/getyourguide/openapi/validation/core/OpenApiRequestValidator.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import com.atlassian.oai.validator.model.Request;
44
import com.atlassian.oai.validator.model.SimpleRequest;
55
import com.atlassian.oai.validator.model.SimpleResponse;
6+
import com.getyourguide.openapi.validation.api.log.LogLevel;
67
import com.getyourguide.openapi.validation.api.log.OpenApiViolationHandler;
78
import com.getyourguide.openapi.validation.api.metrics.MetricsReporter;
89
import com.getyourguide.openapi.validation.api.model.Direction;
@@ -98,7 +99,7 @@ public List<OpenApiViolation> validateRequestObject(
9899
var result = validator.validateRequest(simpleRequest);
99100
var violations = mapper.map(result, request, response, Direction.REQUEST, requestBody);
100101
return violations.stream()
101-
.filter(violation -> !violationExclusions.isExcluded(violation))
102+
.filter(violation -> !LogLevel.IGNORE.equals(violation.getLevel()) && !violationExclusions.isExcluded(violation))
102103
.toList();
103104
} catch (Exception e) {
104105
log.error("[OpenAPI Validation] Could not validate request", e);
@@ -145,7 +146,7 @@ public List<OpenApiViolation> validateResponseObject(
145146
);
146147
var violations = mapper.map(result, request, response, Direction.RESPONSE, responseBody);
147148
return violations.stream()
148-
.filter(violation -> !violationExclusions.isExcluded(violation))
149+
.filter(violation -> !LogLevel.IGNORE.equals(violation.getLevel()) && !violationExclusions.isExcluded(violation))
149150
.toList();
150151
} catch (Exception e) {
151152
log.error("[OpenAPI Validation] Could not validate response", e);

0 commit comments

Comments
 (0)