Skip to content

Commit 0083ef4

Browse files
authored
🚀 Release: beta → master (#79)
* Refactor code structure for improved readability and maintainability * feat(ci): add guard against forbidden docker exec curl usage and improve health checks * fix(ci): enhance guard against forbidden docker exec curl usage in workflows * fix(ci): refine guard against forbidden docker exec curl usage in workflows * feat: enhance CI/CD workflow with production simulation and infra contract checks - Added a new job `production-simulation` to validate the image under production-like conditions. - Implemented infra contract naming guard to enforce canonical naming conventions for network and Redis URLs. - Updated existing checks in the CI workflow to ensure no forbidden patterns are present in deploy paths. - Enhanced `deploy.sh` script to track phase-aware deploy results for better rollback decisions. - Modified readiness checks in `vps-readiness-check.sh` to ensure nginx and Redis are reachable before deployment. - Updated documentation in `infra-contract.md` to reflect new naming conventions and requirements. * fix: enforce canonical redis URL in env.example + scope guard to production paths only Made-with: Cursor * fix(ci): docker-exec curl guard ignores path-prefixed lines and self-doc Made-with: Cursor * fix(ci): remove api-ci-test container before docker rmi in bootstrap step Made-with: Cursor * fix(ci): infra leakage grep pipeline must not fail on zero matches (pipefail) Made-with: Cursor
1 parent bd7b4f1 commit 0083ef4

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

‎.github/workflows/deploy.yml‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -241,8 +241,10 @@ jobs:
241241
# i.e., direct references to alertmanager, loki push clients, or
242242
# docker-compose.monitoring in the application source.
243243
# Exclude comment-only lines (-h suppresses filenames for grep -Ev).
244+
# With bash -o pipefail (GHA default), grep exits 1 when there are zero matches;
245+
# that must not fail the step — only non-empty LEAKS after filtering is an error.
244246
LEAKS=$(grep -rhE "(alertmanager|docker-compose\.monitoring)" src/ tests/ 2>/dev/null \
245-
| grep -Ev '^\s*(//|#|\*|/\*)')
247+
| grep -Ev '^\s*(//|#|\*|/\*)' || true)
246248
if [ -n "$LEAKS" ]; then
247249
echo "::error::Infra client references found in src/ or tests/"
248250
echo "$LEAKS"

0 commit comments

Comments
 (0)