mirror of
https://github.com/hanzoai/replicate.git
synced 2026-10-30 18:23:20 +00:00
* fix(replicate): fail-closed age encryption (no plaintext LTX to S3)
Replicate silently wrote plaintext LTX1 to the destination whenever an age
recipient was unset -- fleet-wide plaintext SQLite backups. This makes
encryption fail-closed: when a replica is told it must encrypt, the absence
of a recipient REFUSES the write (ErrEncryptionRequired) instead of
downgrading to plaintext.
- Replica.RequireEncryption + guards in Start(), WriteLTXFile(), and Compactor
.Compact() -- every write path funnels through a fail-closed check.
- DB.init() fails closed at the earliest point (before the "no db file yet"
early return) so a misconfigured deployment surfaces the error on its first
Sync, loudly, every interval -- never silently no-ops until data appears.
- DB.Snapshot() now routes through Replica.WriteLTXFile (was raw Client), so
snapshots are age-encrypted like L0 sync; previously a plaintext bypass.
- internal.ExtractLTXTimestamp: sniff the age v1 intro so encrypted objects
fall back to write time instead of every backend failing to parse an LTX
header out of ciphertext. All 8 replica clients use it.
- AutoReplicate / CLI: RequireEncryption defaults on; opt out only via explicit
REPLICATE_ALLOW_PLAINTEXT / age.allow-plaintext for non-sensitive dev targets.
Tests: replica_failclosed_test.go (unit: write/start/compact fail closed;
encrypt+round-trip with recipient) and replicate_e2e_encrypt_test.go (full DB
stack: every destination object is age ciphertext across L0 sync AND the
snapshot path; live-writer restore round-trip; DB sync fails closed).
Re-applies the fix from #6 onto a fresh branch off main (#6's branch shares no
git history with main and cannot merge). age bumped 1.4.0->1.6.0 to match the
platform (cloud) and resolve main's broken [email protected] go.sum checksum; go mod
tidy prunes the orphaned prometheus indirect deps left by the metric migration.
* test(replicate): define testutil shim so the package test binary compiles
The luxfi/metric migration (commit 7c8ef84) dropped prometheus but left
db_internal_test.go referencing an undefined `testutil`, so `go test ./` did
not compile at all on main. This adds the minimal in-tree ToFloat64 shim
(luxfi/metric Counter/Gauge expose Get() float64) and drops the now-unused
metric import, unblocking the package test binary.
NOTE: this only fixes COMPILATION. The metric value-assertion tests
(TestDB_*Metrics) remain pre-existing RED because luxfi/metric's
WithLabelValues/With are non-memoizing (proven: vec.WithLabelValues("x").Set(5)
then vec.WithLabelValues("x").Get() == 0). That is an orthogonal, incomplete
metric-migration defect (also affects runtime aggregation at db.go checkpoint
Inc) and must be completed in a dedicated metric PR -- not folded into the
fail-closed security fix. Identical failures occur with or without encryption.
* docs(deps): justify luxfi/age v1.4.0->v1.6.0 bump (forced by broken main go.sum)
* fix(ci): unpoison luxfi/zap + run root metric tests under -tags metrics
Two pre-existing main breakages surfaced by CI (not by the fail-closed
change):
1. luxfi/zap v0.8.1's tag was re-published upstream, so its go.sum hash
no longer verifies (SECURITY ERROR on every job). Bump to the latest
patch in-minor, v0.8.12 (sumdb-verified) — same remedy as the age bump,
no lazy major.
2. luxfi/metric defaults to a NO-OP registry (//go:build !metrics); the
real in-process registry is behind `metrics`. `go test .` ran without
it, so every WithLabelValues returned a throwaway gauge that reads 0 —
the 8 TestDB_*Metrics failures. Root cause is the build tag (not
'non-memoizing' as first guessed, nor a prod defect): the DB's cached
gauge holds the value; only re-derivations under the no-op build miss.
Run the root suite with -tags metrics; passes 8/8 (27s, full package).
* revert(ci): drop -tags metrics test kludge
Prod builds cmd/replicate without -tags metrics (Dockerfile), so the
metric registry is no-op in production regardless; adding the tag only to
CI tests validated a mode prod never runs. The 8 TestDB_*Metrics failures
are a pre-existing main issue (luxfi/metric's no-op default), tracked
separately — not papered over with a test-only build tag. The essential
fix in this PR stands: luxfi/zap v0.8.1->v0.8.12 (re-published-tag go.sum
poison). One way, no kludge.
400 lines
11 KiB
Go
400 lines
11 KiB
Go
package sftp
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"fmt"
|
|
"io"
|
|
"log/slog"
|
|
"net"
|
|
"net/url"
|
|
"os"
|
|
"path"
|
|
"sync"
|
|
"time"
|
|
|
|
"github.com/hanzoai/ltx"
|
|
"github.com/pkg/sftp"
|
|
"golang.org/x/crypto/ssh"
|
|
|
|
"github.com/hanzoai/replicate"
|
|
"github.com/hanzoai/replicate/internal"
|
|
)
|
|
|
|
func init() {
|
|
replicate.RegisterReplicaClientFactory("sftp", NewReplicaClientFromURL)
|
|
}
|
|
|
|
// ReplicaClientType is the client type for this package.
|
|
const ReplicaClientType = "sftp"
|
|
|
|
// Default settings for replica client.
|
|
const (
|
|
DefaultDialTimeout = 30 * time.Second
|
|
)
|
|
|
|
var _ replicate.ReplicaClient = (*ReplicaClient)(nil)
|
|
|
|
// ReplicaClient is a client for writing LTX files over SFTP.
|
|
type ReplicaClient struct {
|
|
mu sync.Mutex
|
|
sshClient *ssh.Client
|
|
sftpClient *sftp.Client
|
|
logger *slog.Logger
|
|
|
|
// SFTP connection info
|
|
Host string
|
|
User string
|
|
Password string
|
|
Path string
|
|
KeyPath string
|
|
HostKey string
|
|
DialTimeout time.Duration
|
|
|
|
// ConcurrentWrites enables concurrent writes for better performance.
|
|
// Note: This makes resuming failed transfers unsafe.
|
|
ConcurrentWrites bool
|
|
}
|
|
|
|
// NewReplicaClient returns a new instance of ReplicaClient.
|
|
func NewReplicaClient() *ReplicaClient {
|
|
return &ReplicaClient{
|
|
logger: slog.Default().WithGroup(ReplicaClientType),
|
|
DialTimeout: DefaultDialTimeout,
|
|
ConcurrentWrites: true, // Default to true for better performance
|
|
}
|
|
}
|
|
|
|
func (c *ReplicaClient) SetLogger(logger *slog.Logger) {
|
|
c.logger = logger.WithGroup(ReplicaClientType)
|
|
}
|
|
|
|
// NewReplicaClientFromURL creates a new ReplicaClient from URL components.
|
|
// This is used by the replica client factory registration.
|
|
// URL format: sftp://[user[:password]@]host[:port]/path
|
|
func NewReplicaClientFromURL(scheme, host, urlPath string, query url.Values, userinfo *url.Userinfo) (replicate.ReplicaClient, error) {
|
|
client := NewReplicaClient()
|
|
|
|
// Extract credentials from userinfo
|
|
if userinfo != nil {
|
|
client.User = userinfo.Username()
|
|
client.Password, _ = userinfo.Password()
|
|
}
|
|
|
|
client.Host = host
|
|
client.Path = urlPath
|
|
|
|
if client.Host == "" {
|
|
return nil, fmt.Errorf("host required for sftp replica URL")
|
|
}
|
|
if client.User == "" {
|
|
return nil, fmt.Errorf("user required for sftp replica URL")
|
|
}
|
|
|
|
return client, nil
|
|
}
|
|
|
|
// Type returns "sftp" as the client type.
|
|
func (c *ReplicaClient) Type() string {
|
|
return ReplicaClientType
|
|
}
|
|
|
|
// Init initializes the connection to SFTP. No-op if already initialized.
|
|
func (c *ReplicaClient) Init(ctx context.Context) error {
|
|
_, err := c.init(ctx)
|
|
return err
|
|
}
|
|
|
|
// init initializes the connection and returns the SFTP client.
|
|
func (c *ReplicaClient) init(ctx context.Context) (_ *sftp.Client, err error) {
|
|
c.mu.Lock()
|
|
defer c.mu.Unlock()
|
|
|
|
if c.sftpClient != nil {
|
|
return c.sftpClient, nil
|
|
}
|
|
|
|
if c.User == "" {
|
|
return nil, fmt.Errorf("sftp user required")
|
|
}
|
|
|
|
// Build SSH configuration & auth methods
|
|
var hostkey ssh.HostKeyCallback
|
|
if c.HostKey != "" {
|
|
var pubkey, _, _, _, err = ssh.ParseAuthorizedKey([]byte(c.HostKey))
|
|
if err != nil {
|
|
return nil, fmt.Errorf("cannot parse sftp host key: %w", err)
|
|
}
|
|
hostkey = ssh.FixedHostKey(pubkey)
|
|
} else {
|
|
slog.Warn("sftp host key not verified", "host", c.Host)
|
|
hostkey = ssh.InsecureIgnoreHostKey()
|
|
}
|
|
config := &ssh.ClientConfig{
|
|
User: c.User,
|
|
HostKeyCallback: hostkey,
|
|
BannerCallback: ssh.BannerDisplayStderr(),
|
|
}
|
|
if c.Password != "" {
|
|
config.Auth = append(config.Auth, ssh.Password(c.Password))
|
|
}
|
|
|
|
if c.KeyPath != "" {
|
|
buf, err := os.ReadFile(c.KeyPath)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("sftp: cannot read sftp key path: %w", err)
|
|
}
|
|
|
|
signer, err := ssh.ParsePrivateKey(buf)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("sftp: cannot parse sftp key path: %w", err)
|
|
}
|
|
config.Auth = append(config.Auth, ssh.PublicKeys(signer))
|
|
}
|
|
|
|
// Append standard port, if necessary.
|
|
host := c.Host
|
|
if _, _, err := net.SplitHostPort(c.Host); err != nil {
|
|
host = net.JoinHostPort(c.Host, "22")
|
|
}
|
|
|
|
// Connect via SSH.
|
|
if c.sshClient, err = ssh.Dial("tcp", host, config); err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
// Wrap connection with an SFTP client.
|
|
// Configure options based on client settings
|
|
opts := []sftp.ClientOption{}
|
|
if c.ConcurrentWrites {
|
|
opts = append(opts, sftp.UseConcurrentWrites(true))
|
|
}
|
|
|
|
if c.sftpClient, err = sftp.NewClient(c.sshClient, opts...); err != nil {
|
|
c.sshClient.Close()
|
|
c.sshClient = nil
|
|
return nil, err
|
|
}
|
|
|
|
return c.sftpClient, nil
|
|
}
|
|
|
|
// DeleteAll deletes all LTX files.
|
|
func (c *ReplicaClient) DeleteAll(ctx context.Context) (err error) {
|
|
defer func() { c.resetOnConnError(err) }()
|
|
|
|
sftpClient, err := c.init(ctx)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
var dirs []string
|
|
walker := sftpClient.Walk(c.Path)
|
|
for walker.Step() {
|
|
if err := walker.Err(); os.IsNotExist(err) {
|
|
continue
|
|
} else if err != nil {
|
|
return fmt.Errorf("sftp: cannot walk path %q: %w", walker.Path(), err)
|
|
}
|
|
if walker.Stat().IsDir() {
|
|
dirs = append(dirs, walker.Path())
|
|
continue
|
|
}
|
|
|
|
if err := sftpClient.Remove(walker.Path()); err != nil && !os.IsNotExist(err) {
|
|
return fmt.Errorf("sftp: cannot delete file %q: %w", walker.Path(), err)
|
|
}
|
|
|
|
internal.OperationTotalCounterVec.WithLabelValues(ReplicaClientType, "DELETE").Inc()
|
|
}
|
|
|
|
// Remove directories in reverse order after they have been emptied.
|
|
for i := len(dirs) - 1; i >= 0; i-- {
|
|
filename := dirs[i]
|
|
if err := sftpClient.RemoveDirectory(filename); err != nil && !os.IsNotExist(err) {
|
|
return fmt.Errorf("sftp: cannot delete directory %q: %w", filename, err)
|
|
}
|
|
}
|
|
|
|
// log.Printf("%s(%s): retainer: deleting all", r.db.Path(), r.Name())
|
|
|
|
return nil
|
|
}
|
|
|
|
// LTXFiles returns an iterator over all available LTX files for a level.
|
|
// SFTP uses file ModTime for timestamps, which is set via Chtimes() to preserve original timestamp.
|
|
// The useMetadata parameter is ignored since ModTime always contains the accurate timestamp.
|
|
func (c *ReplicaClient) LTXFiles(ctx context.Context, level int, seek ltx.TXID, _ bool) (_ ltx.FileIterator, err error) {
|
|
defer func() { c.resetOnConnError(err) }()
|
|
|
|
sftpClient, err := c.init(ctx)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
dir := replicate.LTXLevelDir(c.Path, level)
|
|
fis, err := sftpClient.ReadDir(dir)
|
|
if os.IsNotExist(err) {
|
|
return ltx.NewFileInfoSliceIterator(nil), nil
|
|
} else if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
// Iterate over every file and convert to metadata.
|
|
infos := make([]*ltx.FileInfo, 0, len(fis))
|
|
for _, fi := range fis {
|
|
minTXID, maxTXID, err := ltx.ParseFilename(path.Base(fi.Name()))
|
|
if err != nil {
|
|
continue
|
|
} else if minTXID < seek {
|
|
continue
|
|
}
|
|
|
|
infos = append(infos, <x.FileInfo{
|
|
Level: level,
|
|
MinTXID: minTXID,
|
|
MaxTXID: maxTXID,
|
|
Size: fi.Size(),
|
|
CreatedAt: fi.ModTime().UTC(), // ModTime contains accurate timestamp from Chtimes()
|
|
})
|
|
}
|
|
|
|
return ltx.NewFileInfoSliceIterator(infos), nil
|
|
}
|
|
|
|
// WriteLTXFile writes a LTX file from rd into a remote file.
|
|
func (c *ReplicaClient) WriteLTXFile(ctx context.Context, level int, minTXID, maxTXID ltx.TXID, rd io.Reader) (info *ltx.FileInfo, err error) {
|
|
defer func() { c.resetOnConnError(err) }()
|
|
|
|
sftpClient, err := c.init(ctx)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
filename := replicate.LTXFilePath(c.Path, level, minTXID, maxTXID)
|
|
|
|
// Extract the LTX-header timestamp (age-encrypted objects fall back to write
|
|
// time) and get a reader that replays the full object for upload.
|
|
timestamp, fullReader, err := internal.ExtractLTXTimestamp(rd)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
if err := sftpClient.MkdirAll(path.Dir(filename)); err != nil {
|
|
return nil, fmt.Errorf("sftp: cannot make parent snapshot directory %q: %w", path.Dir(filename), err)
|
|
}
|
|
|
|
f, err := sftpClient.OpenFile(filename, os.O_WRONLY|os.O_CREATE|os.O_TRUNC)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("sftp: cannot open snapshot file for writing: %w", err)
|
|
}
|
|
defer f.Close()
|
|
|
|
n, err := io.Copy(f, fullReader)
|
|
if err != nil {
|
|
return nil, err
|
|
} else if err := f.Close(); err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
// Set file ModTime to preserve original timestamp
|
|
if err := sftpClient.Chtimes(filename, timestamp, timestamp); err != nil {
|
|
return nil, fmt.Errorf("sftp: cannot set file timestamps: %w", err)
|
|
}
|
|
|
|
internal.OperationTotalCounterVec.WithLabelValues(ReplicaClientType, "PUT").Inc()
|
|
internal.OperationBytesCounterVec.WithLabelValues(ReplicaClientType, "PUT").Add(float64(n))
|
|
|
|
return <x.FileInfo{
|
|
Level: level,
|
|
MinTXID: minTXID,
|
|
MaxTXID: maxTXID,
|
|
Size: n,
|
|
CreatedAt: timestamp,
|
|
}, nil
|
|
}
|
|
|
|
// OpenLTXFile returns a reader for an LTX file.
|
|
// Returns os.ErrNotExist if no matching position is found.
|
|
func (c *ReplicaClient) OpenLTXFile(ctx context.Context, level int, minTXID, maxTXID ltx.TXID, offset, size int64) (_ io.ReadCloser, err error) {
|
|
defer func() { c.resetOnConnError(err) }()
|
|
|
|
sftpClient, err := c.init(ctx)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
filename := replicate.LTXFilePath(c.Path, level, minTXID, maxTXID)
|
|
f, err := sftpClient.OpenFile(filename, os.O_RDONLY)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
if offset > 0 {
|
|
if _, err := f.Seek(offset, io.SeekStart); err != nil {
|
|
return nil, err
|
|
}
|
|
}
|
|
|
|
internal.OperationTotalCounterVec.WithLabelValues(ReplicaClientType, "GET").Inc()
|
|
|
|
if size > 0 {
|
|
return internal.LimitReadCloser(f, size), nil
|
|
}
|
|
return f, nil
|
|
}
|
|
|
|
// DeleteLTXFiles deletes LTX files with at the given positions.
|
|
func (c *ReplicaClient) DeleteLTXFiles(ctx context.Context, a []*ltx.FileInfo) (err error) {
|
|
defer func() { c.resetOnConnError(err) }()
|
|
|
|
sftpClient, err := c.init(ctx)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
for _, info := range a {
|
|
filename := replicate.LTXFilePath(c.Path, info.Level, info.MinTXID, info.MaxTXID)
|
|
|
|
c.logger.Debug("deleting ltx file", "level", info.Level, "minTXID", info.MinTXID, "maxTXID", info.MaxTXID, "path", filename)
|
|
|
|
if err := sftpClient.Remove(filename); err != nil && !os.IsNotExist(err) {
|
|
return fmt.Errorf("sftp: cannot delete ltx file %q: %w", filename, err)
|
|
}
|
|
internal.OperationTotalCounterVec.WithLabelValues(ReplicaClientType, "DELETE").Inc()
|
|
}
|
|
|
|
return nil
|
|
}
|
|
|
|
// Cleanup deletes path & directories after empty.
|
|
func (c *ReplicaClient) Cleanup(ctx context.Context) (err error) {
|
|
defer func() { c.resetOnConnError(err) }()
|
|
|
|
sftpClient, err := c.init(ctx)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
if err := sftpClient.RemoveDirectory(c.Path); err != nil && !os.IsNotExist(err) {
|
|
return fmt.Errorf("sftp: cannot delete path: %w", err)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
// resetOnConnError closes & clears the client if a connection error occurs.
|
|
func (c *ReplicaClient) resetOnConnError(err error) {
|
|
if !errors.Is(err, sftp.ErrSSHFxConnectionLost) {
|
|
return
|
|
}
|
|
|
|
if c.sftpClient != nil {
|
|
c.sftpClient.Close()
|
|
c.sftpClient = nil
|
|
}
|
|
if c.sshClient != nil {
|
|
c.sshClient.Close()
|
|
c.sshClient = nil
|
|
}
|
|
}
|