Skip to content
This repository has been archived by the owner on Jun 4, 2021. It is now read-only.

Commit

Permalink
Merge pull request #161 from google/upstream-1583445779
Browse files Browse the repository at this point in the history
Fix credential parsing
  • Loading branch information
jonjohnsonjr authored Mar 5, 2020
2 parents 040e033 + 820aa43 commit 8a11dc8
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 9 deletions.
14 changes: 7 additions & 7 deletions BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ py_library(
"transport/**/*.py",
]),
deps = [
"@httplib2//:httplib2",
"@oauth2client//:oauth2client",
"@six//:six",
"@httplib2",
"@oauth2client",
"@six",
],
)

Expand Down Expand Up @@ -84,8 +84,8 @@ sh_test(
size = "large",
srcs = ["appender_test.sh"],
data = [
":testenv.sh",
":appender.par",
":testenv.sh",
],
)

Expand All @@ -94,8 +94,8 @@ sh_test(
size = "large",
srcs = ["puller_test.sh"],
data = [
":testenv.sh",
":puller.par",
":testenv.sh",
],
)

Expand All @@ -104,8 +104,8 @@ sh_test(
size = "large",
srcs = ["pusher_test.sh"],
data = [
":testenv.sh",
":pusher.par",
":testenv.sh",
],
)

Expand All @@ -114,8 +114,8 @@ sh_test(
size = "large",
srcs = ["digester_test.sh"],
data = [
":testenv.sh",
":digester.par",
":pusher.par",
":testenv.sh",
],
)
2 changes: 1 addition & 1 deletion client/docker_creds_.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ def Get(self):
raise Exception('Error fetching credential for %s, exit status: %d\n%s' %
(self._name, p.returncode, stdout))

blob = json.loads(stdout)
blob = json.loads(stdout.decode('utf-8'))
logging.info('Successfully obtained Docker credentials.')
return Basic(blob['Username'], blob['Secret']).Get()

Expand Down
1 change: 0 additions & 1 deletion puller_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,6 @@ test_image quay.io/coreos/etcd:latest
# TODO: Error pulling and saving image registry.gitlab.com/mattmoor/test-project/image:latest: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1108)
# test_base registry.gitlab.com/mattmoor/test-project/image:latest python3 python:3


# Test pulling by digest
test_image gcr.io/google-containers/pause@sha256:9ce5316f9752b8347484ab0f6778573af15524124d52b93230b9a0dcc987e73e

Expand Down

0 comments on commit 8a11dc8

Please sign in to comment.