Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support promise style async and use region instead zone #430

Merged
merged 17 commits into from
Apr 25, 2024

Conversation

lihsai0
Copy link
Collaborator

@lihsai0 lihsai0 commented Mar 5, 2024

Including #429(rebased)

  • KODO-18440
  • KODO-19071
  • KODO-19072
  • KODO-19933
  • KODO-19934
  • KODO-19935
  • KODO-19983

@qiniu-prow qiniu-prow bot added the size/XXL label Mar 5, 2024
lihsai0 added 8 commits March 12, 2024 15:01
COMPLETED:
- Add promise style;
- Add Retrier for Endpoints or Regions retrying;
- Improve all test cases in rs.js, form_up.js, resume_up.js;
- Fix multiple bugs. An important is v1 resume upload will not callback
  if resume from the last part;
- fix an error comment
- uncomment some test params for more test cases
- ignore ENOENT error when cleanup files after testing
@lihsai0 lihsai0 force-pushed the refactor/bucket-manager-uploader branch from ceeeb9f to f9645c8 Compare March 12, 2024 07:12
Copy link

codecov bot commented Apr 24, 2024

Codecov Report

Attention: Patch coverage is 93.60000% with 48 lines in your changes are missing coverage. Please review.

Project coverage is 87.52%. Comparing base (f60fecd) to head (eb00eff).

❗ Current head eb00eff differs from pull request most recent head e30bb1f. Consider uploading reports for the commit e30bb1f to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #430      +/-   ##
==========================================
+ Coverage   77.62%   87.52%   +9.89%     
==========================================
  Files          27       33       +6     
  Lines        2159     2180      +21     
  Branches      455      414      -41     
==========================================
+ Hits         1676     1908     +232     
+ Misses        483      272     -211     
Files Coverage Δ
index.js 100.00% <ø> (ø)
qiniu/auth/digest.js 100.00% <100.00%> (ø)
qiniu/httpc/endpoint.js 100.00% <100.00%> (ø)
qiniu/httpc/endpointsProvider.js 100.00% <ø> (ø)
qiniu/httpc/middleware/base.js 85.71% <ø> (ø)
qiniu/httpc/middleware/index.js 100.00% <ø> (ø)
qiniu/httpc/middleware/qiniuAuth.js 100.00% <100.00%> (ø)
qiniu/httpc/middleware/retryDomains.js 88.37% <ø> (ø)
qiniu/httpc/middleware/ua.js 91.66% <ø> (-8.34%) ⬇️
qiniu/httpc/region.js 98.36% <100.00%> (+0.68%) ⬆️
... and 15 more

... and 4 files with indirect coverage changes

@bachue bachue merged commit 1e4e326 into master Apr 25, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants