AbstractFilePart.visitDispositionHeader should honor defined charset, close AsyncHttpClient#738

Stephane Landelle · Stephane Landelle · commit 293b6d2dc77e · 2014-10-13T14:40:39.000+02:00
diff --git a/api/src/main/java/org/asynchttpclient/multipart/AbstractFilePart.java b/api/src/main/java/org/asynchttpclient/multipart/AbstractFilePart.java
@@ -69,7 +69,7 @@ protected void visitDispositionHeader(PartVisitor visitor) throws IOException {
         if (fileName != null) {
             visitor.withBytes(FILE_NAME_BYTES);
             visitor.withByte(QUOTE_BYTE);
-            visitor.withBytes(fileName.getBytes(US_ASCII));
+            visitor.withBytes(fileName.getBytes(getCharset() != null ? getCharset() : US_ASCII));
             visitor.withByte(QUOTE_BYTE);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ protected void visitDispositionHeader(PartVisitor visitor) throws IOException {`
`69`	`69`	`if (fileName != null) {`
`70`	`70`	`visitor.withBytes(FILE_NAME_BYTES);`
`71`	`71`	`visitor.withByte(QUOTE_BYTE);`
`72`		`- visitor.withBytes(fileName.getBytes(US_ASCII));`
	`72`	`+ visitor.withBytes(fileName.getBytes(getCharset() != null ? getCharset() : US_ASCII));`
`73`	`73`	`visitor.withByte(QUOTE_BYTE);`
`74`	`74`	`}`
`75`	`75`	`}`