7.0.18 listObjects broken

[PacoDu]

Since 7.0.18 we are experiencing errors with minio client and listObjects. I think the issue was introduced by #877. Also I think there is an issue in the way #877 was merged because xml2js dependency is reintroduced which reverts part of #861

// ...
const res = await minioClient.listObjects(bucket, objectKey, true)

TypeError: xmlobj.Contents.forEach is not a function
    at parseListObjects (/app/node_modules/minio/dist/main/xml-parsers.js:387:21)
    at DestroyableTransform._flush (/app/node_modules/minio/dist/main/transformers.js:88:19)
    at DestroyableTransform.prefinish (/app/node_modules/readable-stream/lib/_stream_transform.js:138:10)
    at DestroyableTransform.emit (events.js:314:20)
    at prefinish (/app/node_modules/readable-stream/lib/_stream_writable.js:619:14)
    at finishMaybe (/app/node_modules/readable-stream/lib/_stream_writable.js:627:5)
    at endWritable (/app/node_modules/readable-stream/lib/_stream_writable.js:638:3)
    at DestroyableTransform.Writable.end (/app/node_modules/readable-stream/lib/_stream_writable.js:594:41)
    at IncomingMessage.onend (_stream_readable.js:683:10)
    at Object.onceWrapper (events.js:420:28)

[ebozduman]

@PacoDu

Yes. This is a regression due to #877. PR #887 fixes it.

issue in the way #877 was merged because xml2js dependency is reintroduced

We still use the faster xml parser, fxp, for parsing and get the benefit of its speed.
We had to bring back xml2js module, because we also need another functionality of it, conversion from json to xml, which does not come with fxp module.

[PacoDu]

@ebozduman Oh ok i see, thanks for the xml2js dependency explanation and the quick fix 🚀 !

[starpit]

Is there a plan to release this fix in a 7.0.19?

[fschuetz04]

Any update on this? The latest version is still 7.0.18, which does not include the mentioned fix. Whenever I use listObjects with a single result, the error occurs.

[prakashsvmx]

@fschuetz04 we are planning for a release soon. Please stay tuned.

[gmacmaster]

Any ideas as to what soon will be?

[prakashsvmx]

@kannappanr