Merge pull request #1352 from cyberark/consume-authn-client-0.19.0

Consume version 0.19.0 of conjur-authn-k8s-client

CHANGELOG.md

@@ -5,13 +5,18 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
-
 ### Added
 - The `vault` provider now supports loading secrets from the KV Version 2 secret
   engine. Reference a secret in Vault using the right path and a field
   navigation in the Secretless configuration.
   [cyberark/secretless-broker#1331](https://github.com/cyberark/secretless-broker/issues/1331)
 
+### Changed
+- Update k8s authenticator client version to
+  [0.19.0](https://github.com/cyberark/conjur-authn-k8s-client/blob/master/CHANGELOG.md#0190---2020-10-08),
+  which adds some fixes around cert injection failure (see also changes in
+  [0.18.1](https://github.com/cyberark/conjur-authn-k8s-client/blob/master/CHANGELOG.md#0181---2020-09-13)).
+
 ## [1.7.0] - 2020-09-11
 
 ### Added

bin/run_integration

@@ -84,7 +84,7 @@ if [[ -x "./stop" ]]; then
 fi
 
 # Translate and copy results unit junit.xml for Jenkins
-rm --force "${test_dir}/junit.xml"
+rm -f "${test_dir}/junit.xml"
 
 docker run --rm \
   --volume "${test_dir}/:/secretless/test/output/" \

go.mod

@@ -6,7 +6,7 @@ require (
 	github.com/codegangsta/cli v1.20.0
 	github.com/containerd/containerd v1.3.2 // indirect
 	github.com/cyberark/conjur-api-go v0.5.2
-	github.com/cyberark/conjur-authn-k8s-client v0.16.1
+	github.com/cyberark/conjur-authn-k8s-client v0.19.0
 	github.com/cyberark/summon v0.7.0
 	github.com/denisenkom/go-mssqldb v0.0.0-20191001013358-cfbb681360f0
 	github.com/docker/distribution v2.7.1+incompatible // indirect
@@ -24,7 +24,6 @@ require (
 	github.com/joho/godotenv v1.2.0
 	github.com/json-iterator/go v1.1.8 // indirect
 	github.com/lib/pq v0.0.0-20180123210206-19c8e9ad0095
-	github.com/modern-go/reflect2 v1.0.1 // indirect
 	github.com/opencontainers/go-digest v1.0.0-rc1 // indirect
 	github.com/opencontainers/image-spec v1.0.1 // indirect
 	github.com/pkg/errors v0.8.1
@@ -34,7 +33,6 @@ require (
 	github.com/spf13/pflag v1.0.5 // indirect
 	github.com/stretchr/testify v1.3.0
 	golang.org/x/crypto v0.0.0-20190510104115-cbcb75029529
-	google.golang.org/appengine v1.4.0 // indirect
 	gopkg.in/yaml.v2 v2.2.2
 	k8s.io/api v0.0.0-20180712090710-2d6f90ab1293
 	k8s.io/apiextensions-apiserver v0.0.0-20180808065829-408db4a50408

go.sum

@@ -29,9 +29,7 @@ github.com/containerd/containerd v1.3.2 h1:ForxmXkA6tPIvffbrDAcPUIB32QgXkt2XFj+F
 github.com/containerd/containerd v1.3.2/go.mod h1:bC6axHOhabU15QhwfG7w5PipXdVtMXFTttgp+kVtyUA=
 github.com/cyberark/conjur-api-go v0.5.2 h1:8ntk07YNRz5bBwjNXkDEAPR70Yr+J2MN8NGlkhaMC3k=
 github.com/cyberark/conjur-api-go v0.5.2/go.mod h1:hwaReWirzgKor+JtH6vbwZaASDXulvd0SzGCloC5uOc=
-github.com/cyberark/conjur-authn-k8s-client v0.13.0/go.mod h1:JTeGIeRO59J7mMEc5yF6FPtk1QnaAzs4GyZa4WldqZc=
-github.com/cyberark/conjur-authn-k8s-client v0.16.1 h1:tiYIUYuBr578BCYlx+mbPiE4dWi+NsCu6Dy9cSRv4/M=
-github.com/cyberark/conjur-authn-k8s-client v0.16.1/go.mod h1:qacUJXCppU1Rg/C+br9B1jBitTq4yG04oc4a+cfI200=
+github.com/cyberark/conjur-authn-k8s-client v0.19.0/go.mod h1:qacUJXCppU1Rg/C+br9B1jBitTq4yG04oc4a+cfI200=
 github.com/cyberark/secretless-broker v1.4.1-0.20191211191712-251c5ec034af/go.mod h1:+GueI3WCJL5gDYaYa38ZokAR8ceEyCVet7MkuZyjf80=
 github.com/cyberark/summon v0.7.0 h1:eBjyNzpJEeFb7BFcOgidM6S/UbEsN7LGSFeQE7szPsk=
 github.com/cyberark/summon v0.7.0/go.mod h1:S7grcxHeUxfL1vRTQUyq9jGK8yG6V/tSlLPQ6tHRO4k=

test/connector/http/basicauth/test

@@ -16,6 +16,8 @@ if $local_test; then
   docker_volume_args=(--volume "$(cd ../../../..; pwd):/secretless")
 fi
 
+TESTS_HAVE_FAILURES="false"
+
 assert_contains() {
   # assert_contains <string> <substring>
   local string=$1
@@ -36,7 +38,7 @@ assert_contains() {
   echo "${substring}"
   echo ""
 
-  exit 1
+  TESTS_HAVE_FAILURES="true"
 }
 
 ping_target_service_thru_secretless() {
@@ -93,4 +95,8 @@ echo "Test: Secretless configured with incorrect password fails"
 bad_resp=$(ping_target_service_thru_secretless 8081)
 assert_contains "$bad_resp" "401 Unauthorized"
 
-echo "PASS"
+if [[ "$TESTS_HAVE_FAILURES" == "true" ]]; then
+  echo "FAIL: basicauth/test"
+  exit 1
+fi
+echo "PASS: basicauth/test"