Skip to content

Remove skipVerify from client struct

Dominic Couture requested to merge remove-skip-verify into master

What does this MR do?

It removes the skipVerify option from the struct in network/client.go

Why was this MR needed?

The option isn't used and SAST complains about it. Removing it prevents accidentally setting InsecureSkipVerify to true.

What's the best way to test this MR?

If the pipeline's green I think we're good?

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitlab-runner/-/issues/27385

Edited by Dominic Couture

Merge request reports