Recognizes non-ASCII filenames in RFC 2231, and suport filename length is zero for multipart/form-data.

[PWZER]

• Suport filename length is zero for multipart/form-data.

  ------sanic
  Content-Disposition: form-data; filename=""; name="test"
  
  ok
  
  ------sanic--
  

• Recognizes non-ASCII filenames in RFC 2231 and RFC 5987, "filename*" format.

  ------sanic
  Content-Disposition: form-data; filename*="utf-8''filename_%C2%A0_test"; name="test"
  
  ok
  
  ------sanic--
  

• Add some test cases in tests/test_requests.py::test_request_multipart_files, like those above.

[codecov]

Codecov Report

Merging #1497 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1497      +/-   ##
==========================================
- Coverage   91.45%   91.41%   -0.04%     
==========================================
  Files          17       17              
  Lines        1731     1735       +4     
  Branches      330      331       +1     
==========================================
+ Hits         1583     1586       +3     
  Misses        123      123              
- Partials       25       26       +1

Impacted Files	Coverage Δ
sanic/request.py	99.52% <100%> (ø)	⬆️
sanic/router.py	95.43% <0%> (-0.46%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52deeba...7b0214d. Read the comment docs.

[harshanarayana]

Since the form_parameters lookup is being done twice, maybe we can move the lookup to a single line and use a variable instead? (this is just a good to have change)

filename_rfc5987 = form_parameters.get("filename*")

if not file_name and filename_rfc5987:
    encoding, _, value = email.utils.decode_rfc5987(filename_rfc2231)
    file_name = unquote(value, encoding=encoding)

Also the format you are using is RFC 5987 standard and not RFC2231 if I remember correctly. (Sorry, for the nitpicking)

[harshanarayana]

Maybe this can be changed to if not file_name? Also, is there a reason to switch and invert the existing check here?