Skip to content

Make Moody's and Slack endpoints reachable for demo capture#156

Merged
kenahrens merged 4 commits into
masterfrom
fix/transactions-stability
Jun 11, 2026
Merged

Make Moody's and Slack endpoints reachable for demo capture#156
kenahrens merged 4 commits into
masterfrom
fix/transactions-stability

Conversation

@kenahrens

Copy link
Copy Markdown
Member

Moody's URL was hardcoded to api.ratings.moodys.com (NXDOMAIN) and Slack defaulted to slack-webhook.example.com (also NXDOMAIN). Both now configurable via env vars with httpbin.org fallbacks so all 15 external APIs generate capturable traffic.

🤖 Generated with Claude Code

kenahrens and others added 4 commits June 11, 2026 08:57
Both transactions-service and accounts-service were crashing under
continuous simulator traffic with 128m heap inside a 512Mi container.
Bump to 384m heap / 1Gi limit. Disable the ErrorSpikeFilter on
transactions to get clean captured traffic for mock building.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The OTel collector scrape config references speedscale-forwarder-metrics
and speedscale-nettap-metrics Services on port 4145 but they didn't
exist, so all Speedscale infra metrics panels in Grafana were empty.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- createTransaction: POST /api/transactions -> /api/transactions/create
  (sim was hitting the controller root which has no POST handler)
- getTransaction: GET /api/transactions/{id} doesn't exist on the
  backend; fall back to list+filter
- check-username/check-email: add to gateway's open endpoints since
  they're called pre-registration without a token

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Moody's URL was hardcoded to api.ratings.moodys.com which is NXDOMAIN.
Slack webhook defaulted to slack-webhook.example.com (also NXDOMAIN).
Both now configurable via env vars (MOODYS_API_URL, SLACK_WEBHOOK_URL)
with httpbin.org fallbacks so traffic is capturable for mock building.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kenahrens kenahrens merged commit a60fc55 into master Jun 11, 2026
16 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