Skip to content

release: Download large files from Google Drive correctly - #39

Merged
kou merged 1 commit into
enactic:mainfrom
kou:release-fix-large-file-download
Sep 9, 2026
Merged

kou merged 1 commit into
enactic:mainfrom
kou:release-fix-large-file-download

Conversation

@kou

@kou kou commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Google Drive returns a "Google Drive can't scan this file for viruses" confirmation HTML page instead of the file itself for large (100MB+) files. The release job saved that page as the file, so some files in the published archive were HTML pages (e.g. 6 casing STL files in 1.1.0 and 2 STEP files of OpenArm Cell for 2.0).

Add "confirm=t" to the download URL to skip the confirmation page.

Also detect download failures instead of silently packaging broken files:

  • Use --fail so that HTTP errors fail the job
  • Fail the job when Google Drive returns an HTML page (deleted file, no permission and so on)
  • Log the size of each downloaded file

Google Drive returns a "Google Drive can't scan this file for viruses"
confirmation HTML page instead of the file itself for large (100MB+)
files. The release job saved that page as the file, so some files in
the published archive were HTML pages (e.g. 6 casing STL files in
1.1.0 and 2 STEP files of OpenArm Cell for 2.0).

Add "confirm=t" to the download URL to skip the confirmation page.

Also detect download failures instead of silently packaging broken
files:

* Use --fail so that HTTP errors fail the job
* Fail the job when Google Drive returns an HTML page (deleted file,
  no permission and so on)
* Log the size of each downloaded file

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@kou
kou merged commit 7f2242b into enactic:main Sep 9, 2026
1 check failed
@kou
kou deleted the release-fix-large-file-download branch September 9, 2026 00:44

This branch had an error being deployed

1 failed deployment
release — e266262b Deployed Sep 9, 2026 by kou via Publish #43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant