Skip to content

chore: pass accept header to asset download request#534

Merged
rjan90 merged 2 commits intomasterfrom
accept-header
Jul 21, 2025
Merged

chore: pass accept header to asset download request#534
rjan90 merged 2 commits intomasterfrom
accept-header

Conversation

@galargh
Copy link
Copy Markdown
Contributor

@galargh galargh commented Jul 21, 2025

This is a follow-up to filecoin-project/lotus#13223

It passes Accept:application/octet-stream header in a request which downloads the release assets. This will ensure we can actually download the assets instead of having to build them from source every time.

@galargh galargh requested review from BigLep, Copilot, rjan90 and rvagg July 21, 2025 13:47
@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz Jul 21, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds an Accept header to asset download requests to ensure proper binary asset retrieval from GitHub releases. The change prevents fallback to building assets from source by ensuring the download request explicitly requests binary content.

  • Adds Accept:application/octet-stream header to curl command for downloading release assets

Comment thread install-filcrypto Outdated
Co-authored-by: Piotr Galar <piotr.galar@gmail.com>
@github-project-automation github-project-automation bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz Jul 21, 2025
@rjan90 rjan90 merged commit dde8f2f into master Jul 21, 2025
5 checks passed
@rjan90 rjan90 deleted the accept-header branch July 21, 2025 19:48
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

4 participants