Skip to content

MMA-10561 Update version check to use CDN#349

Merged
LaurentiuTeodorescu merged 4 commits intomasterfrom
MMA-10561
Mar 18, 2026
Merged

MMA-10561 Update version check to use CDN#349
LaurentiuTeodorescu merged 4 commits intomasterfrom
MMA-10561

Conversation

@dbistriceanu
Copy link
Copy Markdown
Contributor

No description provided.

@LaurentiuTeodorescu LaurentiuTeodorescu marked this pull request as ready for review March 18, 2026 12:12
Copilot AI review requested due to automatic review settings March 18, 2026 12:12
@LaurentiuTeodorescu LaurentiuTeodorescu self-requested a review March 18, 2026 12:14
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

Updates the add-on’s version/update checks to use the CDN endpoint (and HTTPS) and refreshes the documentation link in the README.

Changes:

  • Switched the PHP version-check endpoint to the CDN URL over HTTPS.
  • Switched installer scripts’ version-check base URL to HTTPS.
  • Updated the README documentation link to the new N-able documentation page.

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
library/SpamFilter/Version.php Points update/version check calls at the CDN endpoint using HTTPS.
bin/installer/installer.sh Uses HTTPS for the version-check base URL used by the installer.
bin/installer/old_installer.sh Uses HTTPS for the version-check base URL in the legacy installer script.
README.md Updates the installation/documentation link.

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

@LaurentiuTeodorescu LaurentiuTeodorescu merged commit 5673378 into master Mar 18, 2026
7 checks passed
@LaurentiuTeodorescu LaurentiuTeodorescu deleted the MMA-10561 branch March 18, 2026 12:15
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.

4 participants