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
Fix/warn file too big #8033
Fix/warn file too big #8033
Conversation
98f0294
to
9286bb2
Compare
Codecov Report
@@ Coverage Diff @@
## master #8033 +/- ##
==========================================
+ Coverage 32.71% 32.72% +0.01%
==========================================
Files 1194 1196 +2
Lines 12969 13007 +38
Branches 1280 1285 +5
==========================================
+ Hits 4243 4257 +14
- Misses 7886 7905 +19
- Partials 840 845 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Signed-off-by: Pierre Noël <petersg83@gmail.com>
Signed-off-by: Pierre Noël <petersg83@gmail.com>
Signed-off-by: Pierre Noël <petersg83@gmail.com>
9286bb2
to
878a8bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any changes to the aws and rackspace providers to handle this ? any reason ?
@@ -0,0 +1,8 @@ | |||
const errorTypes = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would move that to the errors file and only expose the errors interface
Signed-off-by: soupette <cyril.lpz@gmail.com>
@petersg83 seems like the linter is happy now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last comment :)
* handle fileTooBig errors Signed-off-by: Pierre Noël <petersg83@gmail.com> * add entityTooLarge error in provider plugins Signed-off-by: Pierre Noël <petersg83@gmail.com> * fix linter Signed-off-by: Pierre Noël <petersg83@gmail.com> * refacto Signed-off-by: Pierre Noël <petersg83@gmail.com> * Add better error message for 413 errors in ML Signed-off-by: soupette <cyril.lpz@gmail.com> * refacto Signed-off-by: Pierre Noël <petersg83@gmail.com> * refacto Signed-off-by: Pierre Noël <petersg83@gmail.com> Co-authored-by: soupette <cyril.lpz@gmail.com>
No modification where made for the providers
aws
andrackspace
as they accept files up to 5 Go and the node buffer limit is 2 Go. So I couldn't test files bigger than 5 Go + the error would throw before the provider says it's too large.It will be needed once we have change the buffers for streams.
@soupette @HichamELBSI I tag you because I updated the front part: the linter was not happy. Can you check everything's fine please?