Skip to content

chore: replace pkg_url with https://pkg.postguard.eu/#178

Merged
rubenhensen merged 1 commit into
mainfrom
chore/update-pkg-url
Jun 19, 2026
Merged

chore: replace pkg_url with https://pkg.postguard.eu/#178
rubenhensen merged 1 commit into
mainfrom
chore/update-pkg-url

Conversation

@dobby-coder

@dobby-coder dobby-coder Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

What

Replaces the production PKG URL https://postguard-main.cs.ru.nl/pkg with https://pkg.postguard.eu/, as requested in #177.

  • conf/config.toml — active pkg_url updated (also dropped a stray trailing space on the line).
  • conf/config.dev.toml — the commented-out example pkg_url updated. The active dev value (http://postguard-pkg:8087) is left unchanged.

Trailing-slash hardening

The requested URL ends in a trailing slash (…postguard.eu/). PkgClient::validate_api_key already trims a trailing slash before building its request path, but the startup parameters fetch in rocket() did not — so a trailing slash would have produced https://pkg.postguard.eu//v2/sign/parameters. I added the same trim_end_matches('/') there so the new value is safe regardless of whether the slash is present.

Testing

  • cargo fmt --all -- --check
  • cargo clippy --all-targets -- -D warnings
  • cargo test --all-targets ✅ (125 passing)

Requested by @rubenhensen in #177.

Replace the production PKG URL https://postguard-main.cs.ru.nl/pkg with
https://pkg.postguard.eu/ in conf/config.toml and the commented example
in conf/config.dev.toml.

Since the new URL carries a trailing slash, harden the startup parameter
fetch in rocket() to trim a trailing slash before appending
/v2/sign/parameters (mirroring the existing trim in
PkgClient::validate_api_key), avoiding a // in the request path.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@rubenhensen rubenhensen marked this pull request as ready for review June 19, 2026 14:04
@rubenhensen rubenhensen merged commit cfe66c6 into main Jun 19, 2026
7 checks passed
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.

1 participant