Skip to content

fix(librespeed-rs): use correct service name #9683

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
tremor021 merged 1 commit into from
Dec 5, 2025
Merged

fix(librespeed-rs): use correct service name #9683

tremor021 merged 1 commit into from
Dec 5, 2025

Conversation

@jniles
Copy link
Contributor

@jniles jniles commented Dec 5, 2025

✍️ Description

The correct service name for librespeed is speedtest_rs. This commit fixes the service name across both install and update scripts.

See librespeed/speedtest-rust#36 (comment).

🔗 Related PR / Issue

Related to #9626.

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • ✨ New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

@jniles
fix(librespeed-rs): use correct service name
1bd2b14 
The correct service name for librespeed is `speedtest_rs`.  This commit fixes the
service name across both install and update scripts.

Related to #9626.
@jniles jniles requested a review from a team as a code owner December 5, 2025 16:39
@github-actions github-actions bot added bugfix update script A change that updates a script labels Dec 5, 2025
@tremor021 tremor021 merged commit 1c325f6 into community-scripts:main Dec 5, 2025
2 checks passed
@jniles jniles deleted the fix-librespeed-rust-service-name branch December 5, 2025 17:43
probers1 pushed a commit to probers1/ProxmoxVE that referenced this pull request Dec 5, 2025
@jniles @probers1
fix(librespeed-rs): use correct service name (community-scripts#9683)
164f9c8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MickLesk MickLesk MickLesk approved these changes

@tremor021 tremor021 tremor021 approved these changes

Assignees

No one assigned

Labels

bugfix update script A change that updates a script

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jniles @MickLesk @tremor021