Skip to content

fix: Provider is crashing when insecure=true

Sign in for the full log view
GitHub Actions / golangci failed Oct 20, 2023 in 1s

reviewdog [golangci] report

reported by reviewdog 🐶

Findings (1)

netbox/provider.go|231 col 83| G402: TLS InsecureSkipVerify set true. (gosec)

Filtered Findings (17)

netbox/tenancy/data_netbox_tenancy_contact.go|46 col 22| string Your query returned no results. has 5 occurrences, make it a constant (goconst)
netbox/tenancy/data_netbox_tenancy_contact.go|47 col 4| string Please change your search criteria and try again. has 5 occurrences, make it a constant (goconst)
netbox/tenancy/data_netbox_tenancy_contact.go|49 col 22| string Your query returned more than one result. has 5 occurrences, make it a constant (goconst)
netbox/tenancy/data_netbox_tenancy_contact.go|50 col 4| string Please try a more specific search criteria. has 5 occurrences, make it a constant (goconst)
netbox/dcim/data_netbox_dcim_device_role.go|52 col 22| string Your query returned no results. has 8 occurrences, make it a constant (goconst)
netbox/dcim/data_netbox_dcim_device_role.go|53 col 4| string Please change your search criteria and try again. has 8 occurrences, make it a constant (goconst)
netbox/dcim/data_netbox_dcim_device_role.go|55 col 22| string Your query returned more than one result. has 8 occurrences, make it a constant (goconst)
netbox/dcim/data_netbox_dcim_device_role.go|56 col 4| string Please try a more specific search criteria. has 8 occurrences, make it a constant (goconst)
netbox/ipam/data_netbox_ipam_aggregate.go|56 col 22| string Your query returned no results. has 12 occurrences, make it a constant (goconst)
netbox/ipam/data_netbox_ipam_aggregate.go|57 col 4| string Please change your search criteria and try again. has 12 occurrences, make it a constant (goconst)
netbox/ipam/data_netbox_ipam_aggregate.go|59 col 22| string Your query returned more than one result. has 12 occurrences, make it a constant (goconst)
netbox/ipam/data_netbox_ipam_aggregate.go|60 col 4| string Please try a more specific search criteria. has 12 occurrences, make it a constant (goconst)
netbox/virtualization/data_netbox_virtualization_cluster.go|37 col 42| unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
netbox/virtualization/resource_netbox_virtualization_cluster.go|163 col 46| unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
netbox/virtualization/resource_netbox_virtualization_cluster_group.go|119 col 51| unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
netbox/ipam/util.go|116 col 12| indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
netbox/provider_test.go|26 col 24| unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)

Annotations

Check failure on line 231 in netbox/provider.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] netbox/provider.go#L231

G402: TLS InsecureSkipVerify set true. (gosec)
Raw output
netbox/provider.go:231:83: G402: TLS InsecureSkipVerify set true. (gosec)
		t.Transport.(*http.Transport).TLSClientConfig = &tls.Config{InsecureSkipVerify: insecure}
		                                                                                ^