fixed GetFileContents() to surely return a non-nil error on Fail. (#138)

google · Jun 16, 2020 · 3c2bc80 · 3c2bc80
1 parent a5c371f
commit 3c2bc80
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/gdrive/gdrive.go b/gdrive/gdrive.go
@@ -1015,7 +1015,16 @@ func (gd *GDrive) GetFileContents(f *File) (io.ReadCloser, error) {
 			// Rate-limit the download, if required.
 			return makeLimitedDownloadReader(resp.Body), nil
 		case Fail:
-			return nil, err
+			if resp != nil && resp.Body != nil {
+				defer resp.Body.Close()
+			}
+			if err != nil {
+				return nil, err
+			} else if resp != nil {
+				return nil, fmt.Errorf("%s", resp.Status)
+			} else {
+				return nil, fmt.Errorf("unknown error")
+			}
 		case Retry:
 		}
 	}