cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From asapping99 <...@git.apache.org>
Subject [GitHub] cordova-plugin-file-transfer pull request #160: When "Content-Type" has "mul...
Date Thu, 06 Oct 2016 07:02:35 GMT
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.
    
    …Content-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 <asapping99@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


Mime
View raw message