[
https://issues.apache.org/jira/browse/CB-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15213447#comment-15213447
]
ASF GitHub Bot commented on CB-10771:
-------------------------------------
Github user paulhickman-a365 commented on the pull request:
https://github.com/apache/cordova-plugin-file-transfer/pull/133#issuecomment-202054865
Won't using "params[key]" as the test make it also ignore non-string values
such as 0 or boolean false? Testing for undefined, null and "" explicitly would
be better.
> plugin-file-transfer upload on windows fails when params has a key with a
> null value
> ------------------------------------------------------------------------------------
>
> Key: CB-10771
> URL: https://issues.apache.org/jira/browse/CB-10771
> Project: Apache Cordova
> Issue Type: Bug
> Components: Plugin File Transfer
> Affects Versions: 1.5.0
> Environment: Windows, cordova 6.0.0
> Reporter: Paul Hickman
> Assignee: Raghav Katyal
> Labels: windows
> Original Estimate: 2h
> Remaining Estimate: 2h
>
> In the windows version of plugin-file-transfer, if you set the options such
> that a key inside the params object has a null value: e.g.
> var options = { params: { keyName: null } }
> Then it will fail to upload with an INVALID_URL error because an exception
> occurs trying to convert the key keyName into a transfer chunk.
> This does not cause an error in the ios or android versions.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
