Skip to content
This repository was archived by the owner on Nov 6, 2019. It is now read-only.

Conversation

@andreaperizzato
Copy link

  • added support for BulkDataExchange and FileTransfer API
  • added zip parser with extracts the zip data from a downloadFile response
  • tests of BulkDataExchange and FileTransfer requests
  • tests of the zip parser using a real response obtained from the sandbox server

- Bulk Data Exchange operations are now supported using
‘BulkDataExchange’ as service name
- File Transfer operations are now supported using ‘FileTransfer’ as
service name
- Added tests for both services
- added a zip parser which extracts the zip data from the
multipart/related response of a `downloadFile` request
- tested using a real response obtained in the sandbox environment
- Added examples
- Added service version headers in BulkDataExchange and FileTransfer
services
@benbuckman
Copy link
Owner

@andreaperizzato Thank you for submitting this. Is it possible to make the tests pass? I'd like to merge it at that point.
Thank you (and sorry for the very long delay in reviewing).

@boboci9
Copy link

boboci9 commented May 29, 2017

Hi,

Did anyone manage to get this solution to work? Any report I try to download from ebay just says 'Error occurred while loading the archive' so I think the issue is with the parseResponseZip function because if I just save the result using browser save it works but not through the code after parsing. Any ideas?

Meteor.call("ebay.downloadFile",args, function(err, result){
        console.log("response ", err, result); --> this gives me exactly what it should 
        var zip = parseResponseZip(result);
        blob = new Blob([zip], {type: "octet/stream"});
        saveAs(blob, "report.zip";);
    })

Thanks in advance for your help.

@andreaperizzato
Copy link
Author

Hi @boboci9,
i've been using this PR in production on node for a while and it works well. What is the zip object on your code? Where does it fail?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants