Do not validate HEAD requests (#27)

pboos · web-flow · commit 0ae72072e456 · 2023-08-24T09:44:00.000+02:00
diff --git a/openapi-validation-api/src/main/java/com/getyourguide/openapi/validation/api/selector/DefaultTrafficSelector.java b/openapi-validation-api/src/main/java/com/getyourguide/openapi/validation/api/selector/DefaultTrafficSelector.java
@@ -51,12 +51,14 @@ private boolean isRandomlySelectedBySampleRate() {
     @Override
     public boolean canRequestBeValidated(RequestMetaData request) {
         return !methodEquals(request.getMethod(), "OPTIONS")
+            && !methodEquals(request.getMethod(), "HEAD")
             && isContentTypeSupported(request.getContentType());
     }
 
     @Override
     public boolean canResponseBeValidated(RequestMetaData request, ResponseMetaData response) {
         return !methodEquals(request.getMethod(), "OPTIONS")
+            && !methodEquals(request.getMethod(), "HEAD")
             && isContentTypeSupported(response.getContentType());
     }
 

Original file line number	Diff line number	Diff line change
`@@ -51,12 +51,14 @@ private boolean isRandomlySelectedBySampleRate() {`
`51`	`51`	`@Override`
`52`	`52`	`public boolean canRequestBeValidated(RequestMetaData request) {`
`53`	`53`	`return !methodEquals(request.getMethod(), "OPTIONS")`
	`54`	`+ && !methodEquals(request.getMethod(), "HEAD")`
`54`	`55`	`&& isContentTypeSupported(request.getContentType());`
`55`	`56`	`}`
`56`	`57`
`57`	`58`	`@Override`
`58`	`59`	`public boolean canResponseBeValidated(RequestMetaData request, ResponseMetaData response) {`
`59`	`60`	`return !methodEquals(request.getMethod(), "OPTIONS")`
	`61`	`+ && !methodEquals(request.getMethod(), "HEAD")`
`60`	`62`	`&& isContentTypeSupported(response.getContentType());`
`61`	`63`	`}`
`62`	`64`