Bugs in import

[mike442144]

    keys.forEach((key) => {
      const fileContent = req.files[key].data.toString('utf8');
      const str = `[${fileContent.replace(/}\n(\s)?\{/gm, '},{').replace(/\{"\$oid":"(.+?)"}/gm, '"$1"')}]`;

      try {
        const arr = JSON.parse(str);

        req.collection.insert(arr).then((stats) => {
          res.status(200).send(`${stats.result.n} document(s) inserted`);
        }).catch((err) => {
          console.error(err);

          res.status(500);
        });
      } catch (err) {
        console.error(err);

        res.status(500);
      }
    });

There are two bugs in this segment of code:

• We have an arr and will execute insert operation for every file, what will happen when res...send be called multiple times?
• If error when JSON.parse, it seems the server will not respond to client, because of lacking res...send function being called

Let me fix it and pls merge then. Thanks!

[dozoisch]

I will be happy to review and merge a PR :)

[luisg595]

This solution worked for me
#433 (comment)
@dozoisch