mirror of
https://github.com/mudler/LocalAI.git
synced 2026-09-15 07:39:20 -04:00
* feat(credentials): parse and match download credential rules Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * fix(credentials): keep secrets out of parse errors and tighten URL matching Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * feat(credentials): resolve secrets lazily and authenticate HTTP per hop Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * fix(credentials): redact secrets in nested and store formatting Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * feat(credentials): add registry keychain and oras credential adapters Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * fix(credentials): match repository rules for Docker Hub in the oras adapter Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * feat(downloads): authenticate HTTP downloads and gallery reads from the credentials store Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * feat(oci): authenticate registry pulls, resumes, blobs and cosign from the credentials store Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * feat(cli): load download credentials from --credentials-file Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * docs(credentials): correct the local-network registry rules Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * fix(credentials): keep secrets out of match and YAML parse errors A match that fails to parse is no longer quoted in the Parse error, since it may be a URL with a token in it. Userinfo is detected before the scheme check, so ftp://user:token@host is refused as userinfo, and a match with a query string or fragment is refused because it can never apply and a query string is where signed URLs carry their token. Every YAML decode error is now redacted, not only type errors: quoted scalars such as a secret under a mismatched !!int tag are replaced and unquoted map keys are cut off. Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * fix(downloads): make auth errors name the real cause and never retry unresolved secrets AuthError now appends its cause, so a registry's DENIED or UNAUTHORIZED detail reaches the operator. HTTP auth errors print only the status text in place of the cause, because the downloader builds that cause from the requested URL, which can carry a signed query string. Registry pulls say that docker config credentials were tried too, and a download that carried a caller-provided credential (WithBearerToken, or an explicit authorization on gallery reads) reports that credential as rejected instead of blaming the store. The Range probe for a leftover partial file now returns an unresolved secret as a permanent error, like the download request already did. Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * fix(credentials): keep oras pulls anonymous on a broken docker helper and close bodies When docker config names a credsStore helper that cannot run, the oras credential func now logs at debug and returns no credential, so public pulls keep working as they did before the adapter existed. The transport closes the request body when a rule's secret cannot be resolved, as the RoundTripper contract requires. The redirect spec now uses a custom header rule on the origin, which net/http would not strip on its own, to prove the transport does not carry credentials to the next hop. Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * test(oci): cover FetchImageBlob authentication against a private registry FetchImageBlob now has a spec that pulls a layer blob by digest from a basic-auth registry through the oras credential adapter, and one that shows the same fetch fails when no rule matches. oras only speaks HTTPS here, so the registry serves TLS and the spec points http.DefaultTransport, which retry.DefaultClient falls back to, at the test server's client for its duration instead of adding a transport seam to production code. Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * docs(credentials): document auth error wording, ollama manifests and registry tokens The errors section now lists the registry and provided-credential messages and says the server's reason is appended. ollama:// manifests are fetched without credentials, so only blob downloads use the file. GHCR, Docker Hub and Quay need basic auth with the token as password, and match rules must not carry a query string or fragment. The backend gallery docs and the container troubleshooting section now point to the private sources page. Assisted-by: Claude:claude-opus-5 [Claude Code] Signed-off-by: Ettore Di Giacinto <mudler@localai.io> * fix(credentials): document trusted file path The credentials path comes from operator configuration. Mark the file read with a scoped G304 explanation to resolve the gosec false positive. Assisted-by: Codex:gpt-6 gosec --------- Signed-off-by: Ettore Di Giacinto <mudler@localai.io> Co-authored-by: Ettore Di Giacinto <mudler@localai.io> Co-authored-by: localai-org-maint-bot <306269227+localai-org-maint-bot@users.noreply.github.com>
76 lines
3.2 KiB
Go
76 lines
3.2 KiB
Go
package credentials_test
|
|
|
|
import (
|
|
"errors"
|
|
"net/url"
|
|
|
|
. "github.com/onsi/ginkgo/v2"
|
|
. "github.com/onsi/gomega"
|
|
|
|
"github.com/mudler/LocalAI/pkg/credentials"
|
|
)
|
|
|
|
var _ = Describe("AuthError", Serial, func() {
|
|
cause := errors.New("upstream said no")
|
|
|
|
It("says no rule matches when none does", func() {
|
|
useStore("")
|
|
err := credentials.NewAuthError("https://ghcr.io/acme/img", "ghcr.io/acme/img:latest", 401, cause)
|
|
Expect(err).To(MatchError(ContainSubstring("no credentials rule matches")))
|
|
Expect(err).To(MatchError(ContainSubstring("ghcr.io/acme/img:latest")))
|
|
var authErr *credentials.AuthError
|
|
Expect(errors.As(err, &authErr)).To(BeTrue())
|
|
Expect(authErr.Match).To(BeEmpty())
|
|
Expect(errors.Is(err, cause)).To(BeTrue())
|
|
})
|
|
|
|
It("names the rule the server rejected", func() {
|
|
useStore("- match: ghcr.io/acme\n bearer: wrong\n")
|
|
err := credentials.NewAuthError("https://ghcr.io/acme/img", "ghcr.io/acme/img:latest", 403, cause)
|
|
Expect(err).To(MatchError(ContainSubstring(`credential "ghcr.io/acme" was rejected`)))
|
|
Expect(err.Error()).NotTo(ContainSubstring("wrong"))
|
|
})
|
|
|
|
It("keeps the cause so registry detail survives", func() {
|
|
useStore("")
|
|
err := credentials.NewAuthError("https://ghcr.io/acme/img", "ghcr.io/acme/img:latest", 401, cause)
|
|
Expect(err).To(MatchError(HaveSuffix(": upstream said no")))
|
|
useStore("- match: ghcr.io/acme\n bearer: wrong\n")
|
|
err = credentials.NewAuthError("https://ghcr.io/acme/img", "ghcr.io/acme/img:latest", 403, cause)
|
|
Expect(err).To(MatchError(HaveSuffix(": upstream said no")))
|
|
})
|
|
|
|
It("says docker config was consulted too for a registry", func() {
|
|
useStore("")
|
|
err := credentials.NewRegistryAuthError("https://ghcr.io/acme/img", "ghcr.io/acme/img:latest", 401, cause)
|
|
Expect(err).To(MatchError("authentication required for ghcr.io/acme/img:latest (status 401): no credentials rule matches it and docker config credentials, if any, were not accepted: upstream said no"))
|
|
})
|
|
|
|
It("does not consult the store when the caller chose the credential", func() {
|
|
useStore("- match: https://cdn.example.com\n bearer: store\n")
|
|
u, err := url.Parse("https://cdn.example.com/blob")
|
|
Expect(err).NotTo(HaveOccurred())
|
|
authErr := credentials.HTTPProvidedCredentialError(u, 401, cause)
|
|
Expect(authErr).To(MatchError(ContainSubstring("the provided credential was rejected by https://cdn.example.com/blob (status 401)")))
|
|
Expect(authErr.Error()).NotTo(ContainSubstring("credentials rule"))
|
|
Expect(errors.Is(authErr, cause)).To(BeTrue())
|
|
})
|
|
|
|
It("does not print an HTTP cause, which may quote the signed URL", func() {
|
|
useStore("")
|
|
u, err := url.Parse("https://cdn.example.com/blob?X-Amz-Signature=topsecret")
|
|
Expect(err).NotTo(HaveOccurred())
|
|
leaky := errors.New("failed to download url \"https://cdn.example.com/blob?X-Amz-Signature=topsecret\"")
|
|
authErr := credentials.HTTPAuthError(u, 401, leaky)
|
|
Expect(authErr.Error()).NotTo(ContainSubstring("topsecret"))
|
|
Expect(errors.Is(authErr, leaky)).To(BeTrue())
|
|
})
|
|
|
|
It("keeps signed query strings out of HTTP errors", func() {
|
|
useStore("")
|
|
u, err := url.Parse("https://cdn.example.com/blob?X-Amz-Signature=topsecret")
|
|
Expect(err).NotTo(HaveOccurred())
|
|
Expect(credentials.HTTPAuthError(u, 401, cause).Error()).NotTo(ContainSubstring("topsecret"))
|
|
})
|
|
})
|