Skip to content

Commit 73711ab

Browse files
committed
clean
1 parent 2f3a16d commit 73711ab

File tree

5 files changed

+3
-10
lines changed

5 files changed

+3
-10
lines changed

src/main/java/com/emc/object/s3/jersey/ChecksumRequestFilter.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,6 @@ public void close() throws IOException {
9292
if (generateContentMd5 != null && generateContentMd5) {
9393
// add Content-MD5 (before anything is written to the final stream)
9494
clientRequestContext.getHeaders().add(RestUtil.HEADER_CONTENT_MD5, DatatypeConverter.printBase64Binary(checksum.getByteValue()));
95-
clientRequestContext.getHeaders().add("xxx", DatatypeConverter.printBase64Binary(checksum.getByteValue()));
96-
9795

9896
// need to re-sign request because Content-MD5 is included in the signature!
9997
if (s3Config.getIdentity() != null) {

src/main/java/com/emc/object/s3/jersey/FilterPriorities.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ public class FilterPriorities {
1313
public static final int PRIORITY_CODEC_REQUEST = 1800;
1414
public static final int PRIORITY_AUTHORIZATION = Priorities.AUTHORIZATION; //2000
1515
public static final int PRIORITY_FAULTINJECTION = 2400;
16-
1716
// providers used during response processing (using descending manner)
1817
public static final int PRIORITY_CHECKSUM_RESPONSE = 4500;
1918
public static final int PRIORITY_CODEC_RESPONSE = 4000;

src/main/java/com/emc/object/s3/jersey/S3JerseyClient.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -136,8 +136,8 @@ public S3JerseyClient(S3Config s3Config) {
136136
}
137137

138138
/**
139-
* Provide a specific Jersey ClientHandler implementation (default is ApacheHttpClient4Handler). If you experience
140-
* performance problems, you might try using URLConnectionClientHandler, but note that it will not support the
139+
* Provide a specific Jersey ClientHandler implementation (default is ApacheHttpClient). If you experience
140+
* performance problems, you might try using URLConnectionClient, but note that it will not support the
141141
* Expect: 100-Continue header and upload size is limited to 2GB. Also note that when using that handler, you should
142142
* set the "http.maxConnections" system property to match your thread count (default is only 5).
143143
*/

src/main/java/com/emc/object/s3/jersey/StreamExceptionWriteInterceptor.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package com.emc.object.s3.jersey;
22

3-
import com.emc.object.util.RestUtil;
43
import org.slf4j.Logger;
54
import org.slf4j.LoggerFactory;
65

src/test/java/com/emc/object/s3/WriteTruncationTest.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -162,10 +162,7 @@ void testTruncatedWrite(boolean useApacheClient,
162162
try {
163163
s3Client.putObject(new PutObjectRequest(getTestBucket(), key, badStream).withObjectMetadata(metadata));
164164
Assert.fail("exception in input stream did not throw an exception");
165-
} catch (RuntimeException e) {
166-
// get RC
167-
// Throwable t = e;
168-
// while (t.getCause() != null && t.getCause() != t) t = t.getCause();
165+
} catch (Exception e) {
169166
if (exceptionType == ExceptionType.RuntimeException) {
170167
Assert.assertTrue(e.getCause() instanceof RuntimeException);
171168
} else {

0 commit comments

Comments
 (0)