Fix set nil for 'checkRedirect' when no value exists, tmp fix.
- [X] All tests passed. If this feature is not already covered by the tests, I added new tests.
- [X] All static analysis checks passed.
- [X] This pull request is on the dev branch.
- [X] I used gofmt for formatting the code before submitting the pull request.
This PR is a temp solution for data race found by go tool, it prevent the tool from failing. in curation we have many requests in parallel using the same http client, one of the struct values 'CheckRedirect' is set to nil after each request, as for the usage of the use today with the client, the value is always nil, so there is no need to write any nil value again which cause data race, For better future use of this client to have parallel requests safely (only on post requests or specific config use it) , this should not be changed after each requests but be determined on the Initialization of the client.
@asafambar fixed it - https://github.com/jfrog/jfrog-client-go/commit/ca94b4567e7b5ff200b46d76f27a03e990eeb3b8#diff-98b36a502daf580571eea44435a154850e1ee83f21d7c2b56d9bf976f7b4adc2R185