mirror of
https://github.com/docker/login-action.git
synced 2026-07-28 19:05:20 +00:00
Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 4ff0d3f1cb | |||
| a7569c1208 |
@@ -126,14 +126,8 @@ describe('getOIDCToken', () => {
|
||||
expect(core.info).toHaveBeenCalledWith('Docker Hub OIDC token request rate limited, retrying in 0ms (attempt 1/5)');
|
||||
});
|
||||
|
||||
test('throws Docker Hub OIDC error responses', async () => {
|
||||
postSpy.mockResolvedValue(httpResponse(400, JSON.stringify({error: 'invalid_request', error_description: 'bad connection', error_uri: 'https://docs.docker.com'})));
|
||||
await expect(dockerhub.getOIDCToken('docker.io', 'dbowie')).rejects.toThrow('Docker Hub API: bad status code 400: {"error":"invalid_request","error_description":"bad connection","error_uri":"https://docs.docker.com"}');
|
||||
});
|
||||
|
||||
test('throws rate limited Docker Hub OIDC error response after retries', async () => {
|
||||
postSpy.mockResolvedValue(httpResponse(429, JSON.stringify({error: 'rate_limited', error_description: 'slow down'}), {'retry-after': '0'}));
|
||||
await expect(dockerhub.getOIDCToken('docker.io', 'dbowie')).rejects.toThrow('Docker Hub API: bad status code 429: {"error":"rate_limited","error_description":"slow down"}');
|
||||
expect(postSpy).toHaveBeenCalledTimes(6);
|
||||
test('throws Docker Hub API errors', async () => {
|
||||
postSpy.mockResolvedValue(httpResponse(400, JSON.stringify({description: 'bad connection'})));
|
||||
await expect(dockerhub.getOIDCToken('docker.io', 'dbowie')).rejects.toThrow('Docker Hub API: bad status code 400: bad connection');
|
||||
});
|
||||
});
|
||||
|
||||
+113
-113
File diff suppressed because one or more lines are too long
+3
-3
File diff suppressed because one or more lines are too long
+3
-2
@@ -2396,11 +2396,12 @@ SOFTWARE.
|
||||
|
||||
-----------
|
||||
|
||||
The following npm package may be included in this product:
|
||||
The following npm packages may be included in this product:
|
||||
|
||||
- js-yaml@5.2.1
|
||||
- js-yaml@5.2.2
|
||||
|
||||
This package contains the following license:
|
||||
These packages each contain the following license:
|
||||
|
||||
(The MIT License)
|
||||
|
||||
|
||||
+1
-1
@@ -30,7 +30,7 @@
|
||||
"@docker/actions-toolkit": "^0.94.0",
|
||||
"http-proxy-agent": "^9.1.0",
|
||||
"https-proxy-agent": "^9.1.0",
|
||||
"js-yaml": "^5.2.1",
|
||||
"js-yaml": "^5.2.2",
|
||||
"uuid": "^14.0.1"
|
||||
},
|
||||
"devDependencies": {
|
||||
|
||||
+16
-11
@@ -108,19 +108,24 @@ const handleResponse = async (resp: httpm.HttpClientResponse): Promise<string> =
|
||||
};
|
||||
|
||||
const parseError = (statusCode: number, body: string): Error => {
|
||||
if (body) {
|
||||
let errResp: unknown;
|
||||
try {
|
||||
errResp = JSON.parse(body);
|
||||
} catch {
|
||||
errResp = undefined;
|
||||
}
|
||||
if (errResp !== undefined) {
|
||||
throw new Error(`Docker Hub API: bad status code ${statusCode}: ${JSON.stringify(errResp)}`);
|
||||
}
|
||||
}
|
||||
if (statusCode === 401) {
|
||||
throw new Error(`Docker Hub API: operation not permitted`);
|
||||
}
|
||||
if (body) {
|
||||
const errResp = parseErrorBody(body);
|
||||
for (const k of ['description', 'message', 'detail', 'error']) {
|
||||
if (errResp[k]) {
|
||||
throw new Error(`Docker Hub API: bad status code ${statusCode}: ${errResp[k]}`);
|
||||
}
|
||||
}
|
||||
}
|
||||
throw new Error(`Docker Hub API: bad status code ${statusCode}`);
|
||||
};
|
||||
|
||||
const parseErrorBody = (body: string): Record<string, string> => {
|
||||
try {
|
||||
return <Record<string, string>>JSON.parse(body);
|
||||
} catch {
|
||||
return {};
|
||||
}
|
||||
};
|
||||
|
||||
@@ -3174,7 +3174,7 @@ __metadata:
|
||||
globals: "npm:^17.3.0"
|
||||
http-proxy-agent: "npm:^9.1.0"
|
||||
https-proxy-agent: "npm:^9.1.0"
|
||||
js-yaml: "npm:^5.2.1"
|
||||
js-yaml: "npm:^5.2.2"
|
||||
prettier: "npm:^3.8.1"
|
||||
typescript: "npm:^5.9.3"
|
||||
uuid: "npm:^14.0.1"
|
||||
@@ -4357,6 +4357,17 @@ __metadata:
|
||||
languageName: node
|
||||
linkType: hard
|
||||
|
||||
"js-yaml@npm:^5.2.2":
|
||||
version: 5.2.2
|
||||
resolution: "js-yaml@npm:5.2.2"
|
||||
dependencies:
|
||||
argparse: "npm:^2.0.1"
|
||||
bin:
|
||||
js-yaml: bin/js-yaml.mjs
|
||||
checksum: 10/2b4c2933af12c97e1c4894a4f27fe9b06dab70a64a96bb50624b4429bef6bf11008bde20d868bce52a36784473314efc30078ba6025b58cf7537961e23b1ae9c
|
||||
languageName: node
|
||||
linkType: hard
|
||||
|
||||
"jsbn@npm:1.1.0":
|
||||
version: 1.1.0
|
||||
resolution: "jsbn@npm:1.1.0"
|
||||
|
||||
Reference in New Issue
Block a user