Skip to content

fix(installer): Add NSSM download mirror #639

Merged
ppawlowski merged 1 commit into
mainfrom
fix-nssm-url
Jun 4, 2026
Merged

fix(installer): Add NSSM download mirror #639
ppawlowski merged 1 commit into
mainfrom
fix-nssm-url

Conversation

@ppawlowski

Copy link
Copy Markdown
Contributor

Description

This pull request adds the FlowFuse-hosted NSSM download mirror. Additionally, it adds checksum validation for the downloaded archive before continuing, ensuring both archive consistency and security.

Related Issue(s)

Fixes #638

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

  • Includes a DB migration? -> add the area:migration label

@ppawlowski ppawlowski requested review from Steve-Mcl and hardillb June 1, 2026 15:39
@ppawlowski ppawlowski changed the title fix: Add NSSM download mirror fix(installer): Add NSSM download mirror Jun 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown

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 mitigates frequent 503 errors when installing the FlowFuse Device Agent on Windows (issue #638) by adding a FlowFuse-hosted S3 mirror as the primary NSSM download source, retaining nssm.cc as a fallback, and adding SHA-256 verification of the downloaded archive.

Changes:

  • Add a FlowFuse S3 mirror URL as the preferred NSSM download source, with nssm.cc as fallback.
  • Replace the PowerShell Invoke-WebRequest-based download with a Go net/http implementation that retries each source up to 3 times with linear backoff.
  • Verify the downloaded archive against a pinned SHA-256 checksum and discard partial/corrupt files on failure.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ppawlowski ppawlowski merged commit 5981399 into main Jun 4, 2026
12 checks passed
@ppawlowski ppawlowski deleted the fix-nssm-url branch June 4, 2026 06:33
@ff-gh-actions-bot ff-gh-actions-bot Bot mentioned this pull request Jun 4, 2026
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.

Installer failing fue to NSSM download 503

4 participants