Misc cleanups (#899)

* apiclient: stop logging short-term cookies

* testing: unset KOPIA_PASSWORD in tests, which disrupts subprocesses

internal/apiclient/apiclient.go

@@ -199,7 +199,7 @@ func (t loggingTransport) RoundTrip(req *http.Request) (*http.Response, error) {
 		return nil, errors.Wrap(err, "round-trip error")
 	}
 
-	log(req.Context()).Debugf("%v %v took %v and returned %v with cookies %v", req.Method, req.URL, clock.Since(t0), resp.Status, resp.Cookies())
+	log(req.Context()).Debugf("%v %v took %v and returned %v", req.Method, req.URL, clock.Since(t0), resp.Status)
 
 	return resp, nil
 }

tests/testenv/cli_test_env.go

@@ -78,6 +78,9 @@ func NewCLITest(t *testing.T) *CLITest {
 		t.Skip()
 	}
 
+	// unset environment variables that disrupt tests when passed to subprocesses.
+	os.Unsetenv("KOPIA_PASSWORD")
+
 	configDir := testutil.TempDirectory(t)
 
 	cleanName := strings.ReplaceAll(strings.ReplaceAll(strings.ReplaceAll(