Artillery: Regression in CSV parsing because of csv-parse upgrade.

Created on 2 Jan 2019  路  8Comments  路  Source: artilleryio/artillery

A new version of csv-parse, i.e. 4.3.0 was released on Dec 31, 2018. This version has some validation refinements including one for the from_line option. artillery passes the from_line option as either 1 or 0 depending on whether headers should be skipped or not. This should be changed to use 2 or 1 instead.

Affected line in artillery/lib/commands/run.js:

from_line: typeof payloadSpec.skipHeader === true ? 1 : 0,

This should be changed to:

from_line: typeof payloadSpec.skipHeader === true ? 2 : 1,

Alternatively, package.json should be changed to use

"csv-parse": "4.2.0",

Most helpful comment

Thanks. When can I expect the latest code on npmjs.com?

All 8 comments

Can someone please look at my pull request for this? All artillery scripts using CSV files are broken with the latest version.

Thanks. When can I expect the latest code on npmjs.com?

ping!

This is also effecting me

Note that the bug has been fixed, but the latest build has not been published to npmjs yet.

Any timeframe on when the latest with this fix will be released? @rajeshja

I only raised the pull request to fix the bug. I'm not part of the team that owns artillery, so will look to @hassy to answer that question.

Hello, any updates when this will be released?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

hassy picture hassy  路  4Comments

DanielMSchmidt picture DanielMSchmidt  路  4Comments

damon-kreft picture damon-kreft  路  3Comments

hassy picture hassy  路  3Comments

hassy picture hassy  路  3Comments