-
Notifications
You must be signed in to change notification settings - Fork 18k
mime/multipart: add field Reader.MaxMIMEHeaderSize #70933
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
This PR (HEAD: 177458d) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/637915. Important tips:
|
Message from Ian Lance Taylor: Patch Set 1: Hold+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/637915. |
This constant should be mentioned in the doc of |
If the field is set, it is used instead of maxMIMEHeaderSize constant, allowing to further constraint memory usage when parsing multipart streams. Fixes golang#68889
177458d
to
1b9c1ec
Compare
I updated the PR. Added notes to methods request.MultipartReader and request.ParseMultipartForm. |
This PR (HEAD: 1b9c1ec) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/637915. Important tips:
|
Message from Борис Нагаев: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/637915. |
If the field is set, it is used instead of maxMIMEHeaderSize
constant, allowing to further constraint memory usage when
parsing multipart streams.
Fixes #68889
Fixes #73087
Fixes #26339