Skip to content
Snippets Groups Projects
Commit 1b548c7f authored by Rhys Arkins's avatar Rhys Arkins
Browse files

fix(docker): ECR logging

parent 1c9c9f68
No related branches found
No related tags found
No related merge requests found
...@@ -558,21 +558,23 @@ function getECRAuthToken(region: string, opts: hostRules.HostRule) { ...@@ -558,21 +558,23 @@ function getECRAuthToken(region: string, opts: hostRules.HostRule) {
return new Promise<string>(resolve => { return new Promise<string>(resolve => {
ecr.getAuthorizationToken({}, (err, data) => { ecr.getAuthorizationToken({}, (err, data) => {
if (err) { if (err) {
logger.warn({ err }, 'ECR getAuthorizationToken error'); logger.trace({ err }, 'err');
logger.info('ECR getAuthorizationToken error');
resolve(null); resolve(null);
}
const authorizationToken =
data &&
data.authorizationData &&
data.authorizationData[0] &&
data.authorizationData[0].authorizationToken;
if (authorizationToken) {
resolve(authorizationToken);
} else { } else {
logger.warn( const authorizationToken =
'Could not extract authorizationToken from ECR getAuthorizationToken response' data &&
); data.authorizationData &&
resolve(null); data.authorizationData[0] &&
data.authorizationData[0].authorizationToken;
if (authorizationToken) {
resolve(authorizationToken);
} else {
logger.warn(
'Could not extract authorizationToken from ECR getAuthorizationToken response'
);
resolve(null);
}
} }
}); });
}); });
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment