Does not fail when dealing with a 403 http status code

[shadoWalker89]

Hi,

Great package,
I'm using your package to download files from google drive, and it works great except for the fact when the file i'm trying to download is not public google drive returns an 403 status code.
Despite the 403, the package does not emit the error event nor call the catch() callback

      dl.on('error', (err) => reject(err));
      dl.start().catch(err => reject(err));

Instead it's emitting the end event and passes The following downloadEndedStats

{
  fileName: '10.mp4',
  filePath: 'C:\\Users\\Melek\\AppData\\Roaming\\MyApp\\presentations\\10.mp4',
  totalSize: null,
  incomplete: false,
  onDiskSize: 873039,
  downloadedSize: 873039
}

As a workaround i'm currently within the end callback checking if the totalSize === null to know if something is wrong, but i think that if returning anything other than a 200 range or 300 range in case of redirects (if supported) this should fail

Thanks

[hgouveia]

Hello @shadoWalker89 , thanks for the report,

when it receives a response we check the status and 403 should goes under error, but i think the issue is the same related with #115 and maybe #113 and is happening at the HEAD step inside the getTotalSize(), where we do a pre request to get the file size first

unfortunately i don't know when i will have the time to take a lot into this, but i will try ASAP, since there are multiple reports related,