Skip to content

Commit

Permalink
Merge pull request #1299 from fluxcd/oci-insecure-fix
Browse files Browse the repository at this point in the history
oci: rename `OCIChartRepository.insecure` to `insecureHTTP`
  • Loading branch information
aryan9600 authored Nov 30, 2023
2 parents dc79750 + 6a88d10 commit 99e5fe7
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions internal/helm/repository/oci_chart_repository.go
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,8 @@ type OCIChartRepository struct {
// verifiers is a list of verifiers to use when verifying a chart.
verifiers []oci.Verifier

// insecureHTTP indicates that the chart is hosted on an insecure registry.
insecure bool
// insecureHTTP indicates that the chart is hosted on an insecure HTTP registry.
insecureHTTP bool
}

// OCIChartRepositoryOption is a function that can be passed to NewOCIChartRepository
Expand All @@ -94,7 +94,7 @@ func WithVerifiers(verifiers []oci.Verifier) OCIChartRepositoryOption {

func WithInsecureHTTP() OCIChartRepositoryOption {
return func(r *OCIChartRepository) error {
r.insecure = true
r.insecureHTTP = true
return nil
}
}
Expand Down Expand Up @@ -369,7 +369,7 @@ func (r *OCIChartRepository) VerifyChart(ctx context.Context, chart *repo.ChartV
}

var nameOpts []name.Option
if r.insecure {
if r.insecureHTTP {
nameOpts = append(nameOpts, name.Insecure)
}

Expand Down

0 comments on commit 99e5fe7

Please # to comment.