fix(#440 #441 #442 #443): harden Docker, gate dev settings, expand unit tests - #579
Merged
Markadrian6399 merged 2 commits intoAug 27, 2026
Conversation
…arGateLabs#443): harden Docker, gate dev settings, expand unit tests StellarGateLabs#443 docker-compose.yml - Gate resource limits behind COMPOSE_CPU_LIMIT / COMPOSE_MEMORY_LIMIT / COMPOSE_MEMORY_RESERVATION env vars so prod overrides require no file edits - Add comment block explaining dev defaults and prod sizing guidance StellarGateLabs#442 Dockerfile - Add comment block with instructions to pin rust:1.88-bookworm and debian:bookworm-slim base images to sha256 digests for production builds - Add OCI image description LABEL (non-root uid 1001) - Confirm --no-install-recommends present on apt-get step StellarGateLabs#441 src/ssrf.rs - Add 10 targeted unit tests covering: broadcast, documentation ranges, multicast, CGNAT boundary values, IPv6 multicast/unspecified, is_blocked_ip dispatcher, malformed URL, no-host URL, unresolvable host StellarGateLabs#440 src/metrics.rs - Add 17 targeted unit tests covering: WebhookMetrics counter independence, histogram bucket cumulativeness, boundary/zero/overflow values; AuthMetrics all 4 counters; HorizonMetrics all 5 counters + cursor age overwrite; TrustlineMetrics record_check correctness, state replacement, asset removal, failure isolation, snapshot sort order, last_success_unix; render() trustline output for missing/present assets and check_failures .gitignore - Exclude tests/schema_snapshot.sql (generated, not hand-edited) - Exclude .idea/, .vscode/, editor swap files, OS artifacts
|
@edwardfavour998-code Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#443 docker-compose.yml
#442 Dockerfile
#441 src/ssrf.rs
#440 src/metrics.rs
.gitignore
What and why
Closes #440
Closes #441
Closes #442
Closes #443
How it was verified
cargo testpassescargo fmt --checkandcargo clippy --all-targets -- -D warningspassRisk
codecargo deny check allpasses).env.example)Notes for the reviewer