> + this.utils = checkNotNull(utils);
> + }
> +
> + @Override
> + public HttpRequest filter(HttpRequest request) throws HttpException {
> + request = this.replaceDateHeader(request);
> + request = this.replaceHostHeader(request);
> + utils.logRequest(signatureLog, request, ">>");
> + request = this.signer.sign(request);
> + utils.logRequest(signatureLog, request, "<<");
> + return request;
> + }
> +
> + private HttpRequest replaceDateHeader(HttpRequest request) {
> + request = request.toBuilder().removeHeader(HttpHeaders.DATE).build();
> + request =
> request.toBuilder().replaceHeader(GlacierHeaders.ALTERNATE_DATE,
> timeStampProvider.get()).build();
Can you remove the duplicated toBuilder and build calls, e.g.,
```
request = request.toBuilder()
.removeHeader(...)
.replaceHeader(...)
.build()
```
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-aws/pull/3/files#r12570768
