Skip to content

fix: Backend S3 Config + Frontend Health Check Logging & Stability Improvements#327

Merged
jstevens888 merged 6 commits into
team_dfrom
team_d-james-payment
Mar 28, 2026
Merged

fix: Backend S3 Config + Frontend Health Check Logging & Stability Improvements#327
jstevens888 merged 6 commits into
team_dfrom
team_d-james-payment

Conversation

@jstevens888

Copy link
Copy Markdown
Contributor

Description

This update introduces backend configuration for AWS S3 storage and enhances frontend backend-health validation with improved logging and error handling.

Backend Changes

  • Added S3 bucket configuration:
    • aws.s3.bucket-name=careconnect-uploads-791068988278
  • Enabled LLM feature flag:
    • careconnect.llm.enabled=true
  • Supports upcoming file upload and Bedrock-related integrations

Frontend Changes

  • Refactored _checkBackendHealth() in welcome_page.dart
  • Added detailed debug logging:
    • Base URL
    • Response status code
    • Response body
    • Parsed health status
    • Error output
  • Improved error handling and resilience
  • Ensured loading state is consistently reset using finally
  • Added slight delay to improve UX during health check

Other Updates

  • Incremented app version:
    • 1.0.0+32 → 1.0.0+38

Why This Matters

  • Provides required configuration for S3-backed storage (BD3/BD4 readiness)
  • Improves visibility into backend connectivity issues during app startup
  • Prevents UI from getting stuck in loading state
  • Supports debugging across environments (local, emulator, deployed)

Testing Notes

  • Verified /v1/api/test/health endpoint returns healthy
  • Confirmed logs output correct base URL and response data
  • Validated UI properly reflects backend health status
  • Tested failure scenarios (timeout / non-200 response)

@github-actions

Copy link
Copy Markdown

CareConnect Quality Gate Report

BLOCKED — One or more required checks failed. Fix the issues below before merging.

CareConnect — Security & Quality Analysis Report

Report Header

Field Value
Generated (UTC) 2026-03-28 05:33:31 UTC
Pipeline Run #622
Trigger pull_request
Scan Root .

All timestamps are reported in Coordinated Universal Time (UTC).

Pull Request

Field Value
PR Number #327
PR Author @jstevens888
Source Branch team_d-james-payment
Target Branch team_d

Commit Details

Field Value
Commit SHA 10a79c5 (full)

Legend

Status Meaning
SUCCESS Tool ran and found no violations
FAILURE Tool found one or more violations
DISABLED Tool is not yet configured
Enforced Violations from this tool will block the merge
Advisory Violations are reported but will not block the merge

Tool Results Summary

Tool Category Status Role Findings
trufflehog Secrets Scan FAILURE Enforced 44 finding(s)
gitleaks Secrets Scan FAILURE Enforced 20 finding(s)
flutter_analyze SAST — Flutter FAILURE Enforced 1486 finding(s)
checkstyle SAST — Java FAILURE Enforced 29571 finding(s)
pmd SAST — Java FAILURE Enforced 11251 finding(s)
spotbugs SAST — Java FAILURE Enforced 710 finding(s)
semgrep SAST — Multi FAILURE Enforced 53 finding(s)
pylint SAST — Python FAILURE Enforced 180 finding(s)
bandit SAST — Python FAILURE Enforced 39 finding(s)
htmlhint SAST — Web FAILURE Enforced 623 finding(s)
stylelint SAST — Web FAILURE Enforced 15 finding(s)
dependency_check SCA — Multi SUCCESS Enforced
trivy SCA — Container FAILURE Enforced 39 finding(s)

Full artifact bundle available in the workflow run artifacts.

@jstevens888 jstevens888 merged commit 65da7a8 into team_d Mar 28, 2026
0 of 3 checks passed
@willywil548 willywil548 deleted the team_d-james-payment branch June 6, 2026 17:40
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