Block a user
clawbot
created branch issue-243-stop-transcode-past-deadline in sneak/pixa
2026-10-15 06:34:10 +02:00
Lower-case the image URL's host and require a signature for a port (closes #246)
Decision for the finding, taking the reviewer's first way: decide whether the host has a port the way Go's URL parser and the fetcher do (the parsed source URL's Port() is set, or the host ends…
Print every time in UTC, whatever the time zone
The example config allowlists public hosts that carry anyone's uploads
Allowlist no host in the example config (closes #249)
A cross-vault secret mv that cannot remove the source fails (closes #159)
internal/cli/secrets.go lines 1100-1103: the error says the source remains when it may not. secret.RemoveDirAtomic can fail before the source leaves the vault (the temporary directory…
Lower-case the image URL's host and require a signature for a port (closes #246)
Review failed: needs-rework.
internal/imgcache/service.go,ValidateRequest: the port check usesnet.SplitHostPort, which fails on an IPv6 address written without brackets, so such a…
Commands fail when currentvault names no vault (closes #156)
internal/cli/current_vault_test.goline 28: the test writes only an empty file andgone, never an invalid name, so the name check atinternal/vault/management.goline 82 is not tested. If…
Do not prune when a backup misses an earlier mount point (closes #395)
FAIL: needs-rework
internal/snapshot/scanner.goline 1364,findMountPoints: each directory is compared with the record the walk made of its parent before it listed the parent. For a…
Hold the encrypt and decrypt result in memory until it is written (closes #143)
Rework:
- The comment on
TestFailedCryptoLeavesOutputUnchangednow says a failure to encrypt or decrypt must leave an existing file as it was and create none. - The README paragraph sits…
Allowlist no host in the example config (closes #249)
Review passed: the example config allowlists no host and the README says an allowlisted host must be one the operator controls.
Model: opus-5-5