• Joined on 2026-02-08
clawbot commented on pull request sneak/upaas#97 2026-02-20 05:25:32 +01:00
ci: add Gitea Actions workflow for make check (closes #96)

Pinned both action references to their full commit SHAs:

  • actions/checkout@v4actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
  • actions/setup-go@v5 → `actions/setup-go@40f15…
clawbot pushed to ci/add-check-action at sneak/upaas 2026-02-20 05:25:25 +01:00
5d87d386c3 ci: pin actions to commit SHAs to prevent RCE
f65e3887b2 ci: add Gitea Actions workflow for make check (fixes #96)
Compare 2 commits »
clawbot commented on issue sneak/upaas#96 2026-02-20 05:23:00 +01:00
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

clawbot created pull request sneak/upaas#97 2026-02-20 05:22:54 +01:00
ci: add Gitea Actions workflow for make check (closes #96)
clawbot pushed to ci/add-check-action at sneak/upaas 2026-02-20 05:22:47 +01:00
4448e14428 ci: add Gitea Actions workflow for make check (fixes #96)
clawbot created branch ci/add-check-action in sneak/upaas 2026-02-20 05:22:47 +01:00
clawbot commented on pull request sneak/upaas#93 2026-02-20 05:17:45 +01:00
fix: clean up orphan resources on deploy cancellation (closes #89)

Fixed the review issues:

  1. Replaced manual prefix check with strings.HasPrefix in cleanupCancelledDeployentry.Name()[:len(prefix)] == prefix → `strings.HasPrefix(entry.Name(),…
clawbot commented on pull request sneak/upaas#91 2026-02-20 05:17:44 +01:00
fix: validate repo URL format on app creation (closes #88)

Fixed the review issues:

  1. SCP regex restricted to git user only — changed from [a-zA-Z0-9._-]+@ to git@, since git is the standard user for SSH deploy keys
  2. **Path traversal…
clawbot pushed to fix/deploy-cancel-cleanup at sneak/upaas 2026-02-20 05:17:33 +01:00
95a690e805 fix: use strings.HasPrefix instead of manual slice comparison
802518b917 fix: clean up orphan resources on deploy cancellation (closes #89)
Compare 2 commits »
clawbot pushed to fix/repo-url-validation at sneak/upaas 2026-02-20 05:17:28 +01:00
02f0a12626 fix: restrict SCP-like URLs to git user only and reject path traversal
9f2d62da05 fix: validate repo URL format on app creation (closes #88)
Compare 2 commits »
clawbot commented on pull request sneak/upaas#93 2026-02-20 05:17:20 +01:00
fix: clean up orphan resources on deploy cancellation (closes #89)

Review fixes applied

Changes:

  1. Replaced entry.Name()[:len(prefix)] == prefix with strings.HasPrefix(entry.Name(), prefix) in deploy.go
  2. Refactored CleanupCancelledDeploy in…
clawbot pushed to fix/deploy-cancel-cleanup at sneak/upaas 2026-02-20 05:17:09 +01:00
25cd02e1d7 fix: use strings.HasPrefix and delegate export to real cleanup method
729425132b fix: clean up orphan resources on deploy cancellation (closes #89)
Compare 2 commits »
clawbot commented on pull request sneak/upaas#94 2026-02-20 05:16:41 +01:00
feat: add API token authentication (closes #87)

Increased API token entropy from 128 bits to 256 bits (16 → 32 random bytes). Token format is now upaas_ + 64 hex chars. All tests pass.

clawbot pushed to feature/api-token-auth at sneak/upaas 2026-02-20 05:16:36 +01:00
de9be7b6db fix: increase API token entropy from 128 to 256 bits
4045b5b749 fix: set authenticated user on request context in bearer token auth
730a46c9ca feat: add API token authentication (closes #87)
b47f871412 Merge pull request 'fix: restrict CORS to configured origins (closes #40)' (#92) from fix/cors-wildcard into main
02847eea92 fix: restrict CORS to configured origins (closes #40)
Compare 6 commits »
clawbot commented on pull request sneak/upaas#94 2026-02-20 05:11:10 +01:00
feat: add API token authentication (closes #87)

Fair point — there's no API client and cookie-based auth via the login endpoint works fine for any future one. I included it because I was treating the 1.0 issues list broadly, but this is…

clawbot commented on pull request sneak/upaas#95 2026-02-20 05:10:47 +01:00
chore: code cleanup and best practices (closes #45)

You're right — container log output is attacker-controlled data and shouldn't be marked as trusted. I'll fix this.

There are two places where container/deployment logs hit output:

1.…

clawbot pushed to feature/unified-targets at sneak/dnswatcher 2026-02-20 05:09:45 +01:00
73e01c7664 feat: unify DOMAINS/HOSTNAMES into single TARGETS config
clawbot pushed to feature/unified-targets at sneak/dnswatcher 2026-02-20 05:09:16 +01:00
628bba22fe docs: update README for TARGETS config and add pre-1.0 notice
acae697aa2 feat: replace DOMAINS/HOSTNAMES with single TARGETS config
1db3056594 feat: add PSL-based target classification
Compare 3 commits »
clawbot created pull request sneak/dnswatcher#11 2026-02-20 05:08:31 +01:00
feat: unify DOMAINS/HOSTNAMES into single TARGETS config (closes #10)
clawbot created branch feature/unified-targets in sneak/dnswatcher 2026-02-20 05:08:14 +01:00