Block a user
chore: code cleanup and best practices (closes #45)
Fixed container log sanitization and all lint issues. make check passes clean (0 issues).
Changes:
internal/handlers/app.go:502— Updated nolint comment from misleading "trusted…
ci: add Gitea Actions workflow for make check (closes #96)
feat: add Gitea Actions CI for make check (closes #96)
make check results ✅
All checks pass on branch feat/ci-make-check:
==> Checking formatting...
==> Running linter...
golangci-lint run --config .golangci.yml ./...
==> Running…
feat: add Gitea Actions CI for make check (closes #96)
ci: add Gitea Actions workflow for make check (closes #96)
Pinned both action references to their full commit SHAs:
actions/checkout@v4→actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5actions/setup-go@v5→ `actions/setup-go@40f15…
needs actions for code standard checks
PR opened: #97
Adds .gitea/workflows/check.yml that runs make check (format, lint, test, build) on pushes to main and PRs targeting main. Uses actions/setup-go with version from go.mod…
ci: add Gitea Actions workflow for make check (closes #96)
fix: clean up orphan resources on deploy cancellation (closes #89)
Fixed the review issues:
- Replaced manual prefix check with
strings.HasPrefixincleanupCancelledDeploy—entry.Name()[:len(prefix)] == prefix→ `strings.HasPrefix(entry.Name(),…
fix: validate repo URL format on app creation (closes #88)
Fixed the review issues:
- SCP regex restricted to
gituser only — changed from[a-zA-Z0-9._-]+@togit@, sincegitis the standard user for SSH deploy keys - **Path traversal…
fix: clean up orphan resources on deploy cancellation (closes #89)
Review fixes applied ✅
Changes:
- Replaced
entry.Name()[:len(prefix)] == prefixwithstrings.HasPrefix(entry.Name(), prefix)indeploy.go - Refactored
CleanupCancelledDeployin…