Skip to content

FTPI 1660: Setup Monitoring Service and Tests up to PHP 8.4#91

Open
Abraham-Flutterwave wants to merge 25 commits into
devfrom
FTPI-1660
Open

FTPI 1660: Setup Monitoring Service and Tests up to PHP 8.4#91
Abraham-Flutterwave wants to merge 25 commits into
devfrom
FTPI-1660

Conversation

@Abraham-Flutterwave

Copy link
Copy Markdown
Contributor

feat(monitoring): add monitoring service.

📋 Summary

Signoz Service Telemetry

🧪 Testing

  • Event sent when /health/ready returns status: ok (probe carries x-api-key)
  • Health result cached — two events produce one probe and two POSTs
  • Event dropped and failure recorded when status is degraded
  • Exactly 3 POST attempts on persistent 503, counted as one breaker failure
  • No retry on non-503 errors (verified with 422)
  • Circuit opens after 3 consecutive failures; subsequent sends make zero HTTP calls
  • Successful send resets failure count and open-until timestamp
  • Open circuit shared via PSR cache blocks all HTTP across workers
  • app.created sequence updated: merchant lookup → health probe → event POST
  • All static state reset in setUp/tearDown to prevent cross-test leakage

✅ Checklist

  • Code follows the project coding style
  • All exceptions swallowed in the telemetry path (never breaks payments)
  • No secrets committed — API key remains the %%SIGNOZ_API_KEY%% build placeholder
  • Unit tests added/updated and passing locally
  • Backwards compatible — no public API or constructor signature changes
  • Verified against staging SigNoz instance
  • Changelog entry added

📦 Breaking changes

None. Constructor signature, public methods, and event payloads are unchanged.


Signed-off-by: Abraham Olaobaju abraham@flutterwavego.com

@wiz-f83cb90411

wiz-f83cb90411 Bot commented Jun 12, 2026

Copy link
Copy Markdown

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities -
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations 15 Medium 1 Low 1 Info
SAST Finding SAST Findings -
Software Management Finding Software Management Findings -
Total 15 Medium 1 Low 1 Info

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@Abraham-Flutterwave Abraham-Flutterwave changed the base branch from master to dev June 12, 2026 12:07
@Flutterwave Flutterwave deleted a comment from gitguardian Bot Jun 12, 2026
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.

3 participants