Fix: use proper transport for CheckReadWriteAccess

[matejvasek]

No description provided.

[matejvasek]

/cc @natalieparellano

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38.89%. Comparing base (7ebe5c6) to head (b736a96).
Report is 81 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #289       +/-   ##
===========================================
- Coverage   58.29%   38.89%   -19.39%     
===========================================
  Files          32       36        +4     
  Lines        3694     3294      -400     
===========================================
- Hits         2153     1281      -872     
- Misses       1184     1819      +635     
+ Partials      357      194      -163     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[matejvasek]

No idea why coverage is failing.

[matejvasek]

/cc @jjbustamante

[jjbustamante]

Just for help on reviewing this change.

func GetTransport(insecure bool) http.RoundTripper {
	if insecure {
		return &http.Transport{
			TLSClientConfig: &tls.Config{
				InsecureSkipVerify: true, // #nosec G402
			},
		}
	}
	return http.DefaultTransport
}

If insecure is false, it is exactly like it is today