okhttp3 3.12.x parse Part BUG

Good bug reports include a failing test! Writing a test helps you to isolate and describe the problem, and it helps us to fix it fast. Bug reports without a failing test or reproduction steps are likely to be closed.

Here’s an example test to get you started. https://gist.github.com/swankjesse/981fcae102f513eb13ed

Version :

3.12.x

Error description :

When the MultipartBody.Part.createFormData(String name, @Nullable String filename, RequestBody body) method is called, if the file name contains Chinese, an IllegalArgumentException will be thrown to locate the calling chain:

MultipartBody.Part.createFormData(String name, @Nullable String filename, RequestBody body) 
Headers.of(String... namesAndValues)
Headers.checkValue(String value, String name)

The final code is as follows:

  static void checkValue(String value, String name) {
    if (value == null) throw new NullPointerException("value for name " + name + " == null");
    for (int i = 0, length = value.length(); i < length; i++) {
      char c = value.charAt(i);
      if ((c <= '\u001f' && c != '\t') || c >= '\u007f') {
        throw new IllegalArgumentException(Util.format(
            "Unexpected char %#04x at %d in %s value: %s", (int) c, i, name, value));
      }
    }
  }

As shown above, this code does not handle Chinese correctly.

[valepakh]

Is it possible to backport this to 3.12.x?

[lognaturel]

Thanks so much for this backport, @valepakh, and for the merge, @swankjesse. You’re helping folks collect data in challenging environments (and languages with Unicode characters). 🎉