Block a user
fix: pin all external refs to cryptographic identity (closes #118)
Pipeline sweep: PR is mergeable and CI checks passed. No code review yet — labeling needs-review. Spawning review agent.
CRITICAL: rce in docker build
PR #119 submitted. Changes:
- Pinned Dockerfile base images (
golang:1.25-alpine,alpine:3.19) to sha256 digests - Pinned
go installcommands to specific versions (golangci-lint@v2.10.1,…
fix: pin all external refs to cryptographic identity (closes #118)
clawbot
created branch fix/pin-external-refs-crypto-identity in sneak/upaas
2026-02-20 19:45:10 +01:00
feat: implement TLS certificate inspector (closes #4)
Code Review: TLS Certificate Inspector
LGTM — Approved.
Strengths
- Functional options pattern (
WithTimeout,WithTLSConfig,WithPort) — idiomatic Go - Proper error handling:…
feat: implement TLS certificate inspector (closes #4)
Rebased onto main. Resolved merge conflict in internal/notify/notify.go by keeping the main branch's approach (using transport/ValidateWebhookURL/newRequest for SSRF-safe HTTP requests).…
feat: implement TLS certificate inspector (closes #4)
Pipeline sweep: PR has merge conflicts (mergeable: false). Moving from merge-ready → needs-rebase.
feat: implement iterative DNS resolver (closes #1)
Removed DNS mocking per review feedback
All resolver tests now make real DNS queries against public DNS servers (google.com, cloudflare.com). No mocking.
Changes
- Replaced entire mock…
fix: disable API v1 write methods (closes #112)
Code Review: PR #115 — Disable API v1 Write Methods
Result: ✅ LGTM — Ready for merge
Checklist
- ✅ No linter/test config files modified
- ✅ Write endpoints fully removed…
fix: add CSRF protection to API v1 routes (closes #112)
make check output
==> Checking formatting...
==> Running linter...
golangci-lint run --config .golangci.yml ./...
0 issues.
==> Running tests...
go test -v -race ./...
? git.eeqj.de/sn…
fix: add CSRF protection to API v1 routes (closes #112)
Add API CSRF protection via X-Requested-With header (closes #112)