Skip to content
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

Parsing remote files with download: true always fails, attempts to parse the URL itself #6

Open
mtchllbrrn opened this issue Nov 18, 2015 · 5 comments

Comments

@mtchllbrrn
Copy link

Sample code:

Papa.parse('http://samplecsvs.s3.amazonaws.com/SalesJan2009.csv', {
  download: true,
  complete: function(results) {
    console.log(results);
  }
})

Returns:

{ data: [ [ 'http://samplecsvs.s3.amazonaws.com/SalesJan2009.csv' ] ],
  errors:
   [ { type: 'Delimiter',
       code: 'UndetectableDelimiter',
       message: 'Unable to auto-detect delimiting character; defaulted to \',\'',
       row: undefined } ],
  meta:
   { delimiter: ',',
     linebreak: '\n',
     aborted: false,
     truncated: false } }
@bewallyt
Copy link

I'm getting the same error. Were you able to find a solution?

@mtchllbrrn
Copy link
Author

Unfortunately not. My needs changed shortly after I stumbled upon this bug, which obviated the need to come up with a solution.

@rozklad
Copy link

rozklad commented Apr 19, 2016

looks like it works good on client side, but does what you describe on server side

@allenfuller
Copy link

I keep running into this myself. Have tried both calling a file from S3 and a local file, but either option just returns:

Parsing complete:
{ data: [],
  errors: 
    [ { type: 'Delimiter',
        code: 'UndetectableDelimiter',
        message: 'Unable to auto-detect delimiting character; defaulted to \',\'',
        row: undefined } ],
   meta: 
    { delimiter: ',',
      linebreak: '\n',
      aborted: false,
      truncated: false,
      fields: [ 'http://s3.amazonaws.com/bucket/file_name.csv' ] } }

@harrisonhunter
Copy link
Owner

Added this note to the ReadME but wanted to post here in case anyone is subscribed to this thread:

Note: This package is no longer necessary: Note: If you are using a recent version of Meteor or imports, you no longer need this package. You can install the papaparse npm package directly and import it. In the root of your meteor project run meteor npm install --save papaparse. Then, you can import with import Papa from 'papaparse'; or import * as Papa from 'papaparse';

This problem is likely solved by switching to the npm version of the package OR is a problem in the npm version of the package and is thus out of scope for this repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants