remotestorage.js icon indicating copy to clipboard operation
remotestorage.js copied to clipboard

Google Drive _getFile doesn't check if a file ID exists before using it

Open stuartlangridge opened this issue 4 years ago • 1 comments

In _getFile at https://github.com/remotestorage/remotestorage.js/blob/5b580e8e959a34e7ed2ab89e417929ebb3d04c90/src/googledrive.ts#L476 and https://github.com/remotestorage/remotestorage.js/blob/5b580e8e959a34e7ed2ab89e417929ebb3d04c90/src/googledrive.ts#L477 the Google Drive backend retrieves a file ID but doesn't check that that file ID exists before using it in a request, leading to (if the requested file doesn't exist at all in Google Drive and so has no ID) requests being made to https://www.googleapis.com/drive/v2/files/undefined which is definitely wrong.

This isn't a PR because I'm not sure what should happen here; should that function throw an error, or fake a response object with statusCode 404, or something else?

stuartlangridge avatar Oct 29 '21 16:10 stuartlangridge

This isn't a PR because I'm not sure what should happen here; should that function throw an error, or fake a response object with statusCode 404, or something else?

Faking a 404 seems like a reasonable option to me. I guess any potential issues with that approach would become apparent once you implement and test it.

raucao avatar Oct 31 '21 09:10 raucao