Skip to content
This repository was archived by the owner on Jan 28, 2025. It is now read-only.

fix(lambda-at-edge): add retries for s3 calls #720

Merged
merged 1 commit into from
Oct 28, 2020
Merged

Conversation

dphang
Copy link
Collaborator

@dphang dphang commented Oct 28, 2020

Updates s3-client v3 and adds retries to fix: aws/aws-sdk-js-v3#1196

@codecov
Copy link

codecov bot commented Oct 28, 2020

Codecov Report

Merging #720 into master will decrease coverage by 0.13%.
The diff coverage is 55.55%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #720      +/-   ##
==========================================
- Coverage   79.80%   79.67%   -0.14%     
==========================================
  Files          55       55              
  Lines        1763     1771       +8     
  Branches      378      380       +2     
==========================================
+ Hits         1407     1411       +4     
- Misses        298      302       +4     
  Partials       58       58              
Impacted Files Coverage Δ
...ackages/libs/lambda-at-edge/src/default-handler.ts 93.05% <55.55%> (-1.38%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9a2451c...25c3f9b. Read the comment docs.

@dphang dphang merged commit 72252fc into master Oct 28, 2020
@delete-merged-branch delete-merged-branch bot deleted the dphang/retry-s3 branch October 28, 2020 23:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ECONNRESET exceptions when running in Lambda environment
1 participant