docker/buildx
1
0
Fork 0
mirror of https://gitea.com/Lydanne/buildx.git synced 2025-11-04 10:03:42 +08:00
Code Issues Packages Projects Releases 1 Wiki Activity

Merge pull request #1693 from jedevc/remote-tls-missing-key-cert

Browse Source 
remote: avoid tls error if both key and cert are not set
This commit is contained in:
Tõnis Tiigi
2023-03-23 10:58:12 -07:00
committed by GitHub
parent fd8eaab2df 56876ab825
commit 948414e1b2
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
driver/remote/factory.go
View File

@@ -98,12 +98,12 @@ func (f *factory) New(ctx context.Context, cfg driver.InitConfig) (driver.Driver
if tls.caCert == "" { if tls.caCert == "" {
missing = append(missing, "cacert") missing = append(missing, "cacert")
} }
if tls.cert == "" { if tls.cert != "" && tls.key == "" {
missing = append(missing, "cert")
}
if tls.key == "" {
missing = append(missing, "key") missing = append(missing, "key")
} }
if tls.key != "" && tls.cert == "" {
missing = append(missing, "cert")
}
if len(missing) > 0 { if len(missing) > 0 {
return nil, errors.Errorf("tls enabled, but missing keys %s", strings.Join(missing, ", ")) return nil, errors.Errorf("tls enabled, but missing keys %s", strings.Join(missing, ", "))
} }