Remove noisy SAML verification domain error

Signed-off-by: Bryan Frimin <bryan@getprobo.com>
This commit is contained in:
Bryan Frimin
2026-02-24 09:03:55 +01:00
parent 7ca891abf0
commit 0a0e722c68

View File

@@ -16,6 +16,7 @@ package iam
import ( import (
"context" "context"
"errors"
"fmt" "fmt"
"strings" "strings"
"time" "time"
@@ -42,6 +43,11 @@ const (
txtRecordValuePrefix = "probo-verification=" txtRecordValuePrefix = "probo-verification="
) )
var (
errDomainTXTRecordNotFound = errors.New("domain TXT record not found")
errDomainTXTRecordMismatch = errors.New("domain TXT record mismatch")
)
func NewSAMLDomainVerifier( func NewSAMLDomainVerifier(
pgClient *pg.Client, pgClient *pg.Client,
logger *log.Logger, logger *log.Logger,
@@ -108,10 +114,17 @@ func (v *SAMLDomainVerifier) checkUnverifiedDomains(ctx context.Context) error {
} }
if err := v.tryVerifyDomain(ctx, config.ID); err != nil { if err := v.tryVerifyDomain(ctx, config.ID); err != nil {
if errors.Is(err, errDomainTXTRecordNotFound) || errors.Is(err, errDomainTXTRecordMismatch) {
v.logger.InfoCtx(ctx, "domain verification pending",
log.String("config_id", config.ID.String()),
log.Error(err),
)
} else {
v.logger.ErrorCtx(ctx, "cannot verify domain", v.logger.ErrorCtx(ctx, "cannot verify domain",
log.String("config_id", config.ID.String()), log.String("config_id", config.ID.String()),
log.Error(err), log.Error(err),
) )
}
continue continue
} }
@@ -187,7 +200,7 @@ func (v *SAMLDomainVerifier) checkDNSTXTRecord(emailDomain string, expectedValue
} }
if len(resp.Answer) == 0 { if len(resp.Answer) == 0 {
return fmt.Errorf("cannot find TXT record for %q", emailDomain) return fmt.Errorf("%w for %q", errDomainTXTRecordNotFound, emailDomain)
} }
for _, answer := range resp.Answer { for _, answer := range resp.Answer {
@@ -203,5 +216,5 @@ func (v *SAMLDomainVerifier) checkDNSTXTRecord(emailDomain string, expectedValue
} }
} }
return fmt.Errorf("cannot find matching TXT record for %q", emailDomain) return fmt.Errorf("%w for %q", errDomainTXTRecordMismatch, emailDomain)
} }