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 uploading big files using a browser #956

Closed
korabelnikov opened this issue Dec 13, 2019 · 10 comments · Fixed by #3692
Closed

Support uploading big files using a browser #956

korabelnikov opened this issue Dec 13, 2019 · 10 comments · Fixed by #3692
Assignees
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@korabelnikov
Copy link

I'm trying to upload video of size 2GB and webpage show me error just after few second

Could not create the task
Error: Network Error.

and I've checked it works with videos of size 100mb

@bsekachev
Copy link
Member

@korabelnikov
Hi, what error can you see in the Browser Console?

Try to use share functionality.
Similar issue: #799

@nmanovic nmanovic changed the title can't upload big video Support uploading big files over a browser Dec 16, 2019
@nmanovic nmanovic added duplicate This issue or pull request already exists enhancement New feature or request help wanted labels Dec 16, 2019
@nmanovic nmanovic added this to the Backlog milestone Dec 16, 2019
@nmanovic nmanovic changed the title Support uploading big files over a browser Support uploading big files using a browser Dec 16, 2019
@korabelnikov
Copy link
Author

Failed to load resource: the server responded with a status of 413 (Request Entity Too Large)
create:1 
Access to XMLHttpRequest at 'http://localhost:8080/api/v1/tasks/8/data' from origin 'http://localhost:7080' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
cvat-app.tsx:145 Error: Network Error.

I suppose it's not an enhancement but real bug

@nmanovic
Copy link
Contributor

@korabelnikov , in the particular case your problem isn't connected with the size of data. The request is blocked by CORS policy. There are settings in CVAT which you need to adjust.

# Cross-Origin Resource Sharing settings for CVAT UI
UI_SCHEME = os.environ.get('UI_SCHEME', 'http')
UI_HOST = os.environ.get('UI_HOST', 'localhost')
UI_PORT = os.environ.get('UI_PORT', '3000')
CORS_ALLOW_CREDENTIALS = True
CSRF_TRUSTED_ORIGINS = [UI_HOST]
UI_URL = '{}://{}'.format(UI_SCHEME, UI_HOST)
if len(UI_URL):
    UI_URL += ':{}'.format(UI_PORT)

CORS_ORIGIN_WHITELIST = [UI_URL]
CORS_REPLACE_HTTPS_REFERER = True

@korabelnikov
Copy link
Author

korabelnikov commented Dec 16, 2019

Thank you! Maybe it worth to add this adjustment as default?

@bsekachev
Copy link
Member

@korabelnikov
Actually, default settings from current develop branch are expected to work.
It wasn't stable for some short time, but now it must be.

@korabelnikov
Copy link
Author

@bsekachev I've checkout this repo just a few days ago

@bsekachev
Copy link
Member

@korabelnikov

Anyway. Try rebuld and clean the browser cache just in case.
CORS messages aren't expected to appear in default settings.

What about code 413 (Request Entity Too Large), CVAT never supports large files uploading via a browser. We use "shared" functionality to do it. But I agree it make sense to implement better approach of uploading.

@korabelnikov
Copy link
Author

I get, just simple message that shows "CVAT doesn't support large file uploading, use shared folder functionality instead" will be helpfull

@bsekachev
Copy link
Member

@korabelnikov
Great idea!

@nmanovic
Copy link
Contributor

#3692

@nmanovic nmanovic added bug Something isn't working duplicate This issue or pull request already exists and removed duplicate This issue or pull request already exists enhancement New feature or request help wanted labels Nov 28, 2021
@nmanovic nmanovic removed this from the Backlog milestone Nov 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants