Skip to content

Commit

Permalink
Merge pull request #109 from check-run-reporter/add-timeout
Browse files Browse the repository at this point in the history
add timeout
  • Loading branch information
ianwremmel authored Apr 18, 2022
2 parents aa8d81b + 43be3d4 commit 6266797
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
5 changes: 4 additions & 1 deletion src/commands/submit.ts
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,10 @@ async function tryMultiStepUploadOrFallbackToSingle(
if (axios.isAxiosError(err)) {
// CI doesn't like safe-access here.
if (err.response && err.response.status === 404) {
context.logger.info('Falling back to single step upload');
context.logger.info(
'Received 404 trying to get signed URLs. Assuming feature is notn released yet and falling back to single step upload',
{err}
);
// eslint-disable-next-line import/no-deprecated
return await singleStepUpload(input, context);
}
Expand Down
2 changes: 1 addition & 1 deletion src/lib/upload.ts
Original file line number Diff line number Diff line change
Expand Up @@ -104,8 +104,8 @@ export async function getSignedUploadUrls(
{filenames, label, root, sha},
{
auth: {password: token, username: 'token'},

maxContentLength: Infinity,
timeout: 30000,
}
);

Expand Down

0 comments on commit 6266797

Please sign in to comment.