> @@ -208,6 +211,9 @@ public GeneratedHttpRequest apply(Invocation invocation) {
>        }
>  
>        requestBuilder.filters(getFiltersIfAnnotated(invocation));
> +      if (stripExpectHeader) {
> +          requestBuilder.filter(new StripExpectHeader());
> +      }

Given that this filter is explicitly added here (and not specified inside a 
`@RequestFilters` annotation in the api classes), could you add some unit tests 
to the 
[RestAnnotationProcessorTest](https://github.com/maginatics/jclouds/blob/master/core/src/test/java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java)
 class to make sure filters are properly populated given the different possible 
values of the property?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/65/files#r5128555

Reply via email to