cyberark / secretless-broker

Secure your apps by making them Secretless
Apache License 2.0
234 stars 40 forks source link

Updates conjur-authn-k8s version to v0.19.1 #1375

Closed diverdane closed 3 years ago

diverdane commented 3 years ago

What does this PR do?

This change updates the version of cyberark/conjur-authn-k8s-client that is used by the Secretless Broker to v0.19.1. In this release of cyberark/conjur-authn-k8s-client, the Authenticate and ParseAuthenticationResponse methods were combined into a single method, so calls to the conjur-authn-k8s-client are changed accordingly.

This change also removes a -mod=readonly flag from the macOS keychain test. This flag was causing failures in the macOS GitHub actions test because the macOS builds were requiring changes to go.sum, and having -mod=readonly causes this to be an error condition.

This change also includes a run of go mod tidy to update go.mod and go.sum.

What ticket does this PR close?

Resolves #1368

Checklists

Change log

Test coverage

Documentation

(For releases only) Manual tests

izgeri commented 3 years ago

it might be good to have a changelog message, just because we try to keep our authn-k8s client versions aligned between k8s components when we do suite or DAP releases

codeclimate[bot] commented 3 years ago

Code Climate has analyzed commit bbaf72f0 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 53.6% (0.0% change).

View more on Code Climate.