Skip to content

chore: Use Maven Central mirror and cache for WireMock in Windows CI#260

Merged
keelerm84 merged 1 commit into
mainfrom
mk/SDK-2613/fix-wiremock
Jun 26, 2026
Merged

chore: Use Maven Central mirror and cache for WireMock in Windows CI#260
keelerm84 merged 1 commit into
mainfrom
mk/SDK-2613/fix-wiremock

Conversation

@keelerm84

@keelerm84 keelerm84 commented Jun 26, 2026

Copy link
Copy Markdown
Member

Summary

The Windows CI job downloaded the WireMock standalone jar directly from Maven Central (repo1.maven.org) on every run. Maven Central blocks GitHub's shared runner IPs for automated consumption, returning an HTTP 403 Terms of Service error and failing the build.

This change:

  • Switches the download to the Google-hosted Maven Central mirror (maven-central.storage-download.googleapis.com), which serves the identical artifact and is not subject to the per-IP block.
  • Caches the jar with actions/cache keyed on the WireMock version, so it is only fetched on a cache miss.

The existing checksum verification step is unchanged and now validates both freshly downloaded and cache-restored jars.

A code search across the launchdarkly and launchdarkly-labs orgs confirms php-server-sdk is the only repository downloading the WireMock standalone jar this way, so no other repos need the fix.


Note

Low Risk
CI-only change to artifact download and caching; runtime SDK behavior is unchanged.

Overview
Fixes Windows CI failures when fetching the WireMock standalone JAR from repo1.maven.org (HTTP 403 from Maven Central’s automated-use limits on shared GitHub runner IPs).

The workflow now caches wiremock.jar with actions/cache@v4 (keyed to WireMock 2.31.0) and only runs the download step on a cache miss. The download URL moves to the Google-hosted Maven Central mirror (maven-central.storage-download.googleapis.com) for the same artifact. The existing SHA-256 checksum step still runs for both freshly downloaded and restored jars.

Reviewed by Cursor Bugbot for commit 486b81a. Bugbot is set up for automated code reviews on this repo. Configure here.

The Windows CI job downloaded the WireMock standalone jar from Maven
Central on every run. Maven Central blocks GitHub's shared runner IPs
for automated consumption, returning a 403 Terms of Service error.

Switch the download to the Google-hosted Maven Central mirror, which
serves the identical artifact without that limit, and cache the jar
with actions/cache so it is only fetched on a cache miss. The existing
checksum verification still runs against both fresh and cached jars.
@keelerm84 keelerm84 marked this pull request as ready for review June 26, 2026 19:19
@keelerm84 keelerm84 requested a review from a team as a code owner June 26, 2026 19:19

@aaron-zeisler aaron-zeisler 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.

LGTM

@keelerm84 keelerm84 changed the title fix: Use Maven Central mirror and cache for WireMock in Windows CI chore: Use Maven Central mirror and cache for WireMock in Windows CI Jun 26, 2026
@keelerm84 keelerm84 merged commit 305fb78 into main Jun 26, 2026
20 checks passed
@keelerm84 keelerm84 deleted the mk/SDK-2613/fix-wiremock branch June 26, 2026 20:22
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.

2 participants