You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by asapping99 <gi...@git.apache.org> on 2016/10/06 07:02:35 UTC
[GitHub] cordova-plugin-file-transfer pull request #160: When "Content-Type" has "mul...
GitHub user asapping99 opened a pull request:
https://github.com/apache/cordova-plugin-file-transfer/pull/160
When "Content-Type" has "multipart/form-data", should not contain a "Content-Length" in the boundary area.
<!--
Please make sure the checklist boxes are all checked before submitting the PR. The checklist
is intended as a quick reference, for complete details please see our Contributor Guidelines:
http://cordova.apache.org/contribute/contribute_guidelines.html
Thanks!
-->
### Platforms affected
platform : ios
### What does this PR do?
"https://www.w3.org/Protocols/rfc1341/7_2_Multipart.html" in the article "Multipart Form" of the site, there is no content that ey included the "Content-Length" to the "boundary" region, "boundary "in the region," Content-Type "," Content-Disposition "and it has been seen to have put," Content-Transfer-Encoding "only. Therefore, it is unnecessary source, there is a possibility that the bug occurs, it is assumed that the suit is to delete.
### What testing has been done on this change?
Bug you do not check the "Content-Length" of the boundary in the area occurred in the services currently under development. Once you apply the source, it was Mashi normal operation.
### Checklist
- [ ] [Reported an issue](http://cordova.apache.org/contribute/issues.html) in the JIRA database
- [ ] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
- [ ] Added automated test coverage as appropriate for this change.
\u2026Content-Length" in the boundary area.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/asapping99/cordova-plugin-file-transfer master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cordova-plugin-file-transfer/pull/160.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #160
----
commit 217c1a5ef96caae8ffe7e03f22086a7bc416164a
Author: asapping99 <as...@gmail.com>
Date: 2016-10-06T06:39:39Z
When "Content-Type" has "multipart/form-data", should not contain a "Content-Length" in the boundary area.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org
[GitHub] cordova-plugin-file-transfer issue #160: When "Content-Type" has "multipart/...
Posted by cordova-qa <gi...@git.apache.org>.
Github user cordova-qa commented on the issue:
https://github.com/apache/cordova-plugin-file-transfer/pull/160
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org