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

File upload on files_drop does not abort if tried to cancel after waiting for few seconds #35086

Closed
skshetry opened this issue Apr 24, 2019 · 6 comments · Fixed by #35130
Closed
Assignees
Labels
Milestone

Comments

@skshetry
Copy link
Member

skshetry commented Apr 24, 2019

Regarding #34097:

Canceling upload in files_drop did not work correctly if it is canceled after waiting for a few sec. But, this does work if it is canceled immediately (which didn't work on v10.1.1). The uploading message does disappear after the file is uploaded.

See test video here: https://cloud.owncloud.com/index.php/s/1psr6GzChn3u83L

Originally posted by @skshetry in #35054 (comment)

Steps to reproduce

  1. Share a public link for files_drop.
  2. Upload a file (upload a large file so you have a larger window to cancel upload).
  3. Wait for a few seconds (but not too much that the time you cancel, it's already uploaded).
  4. Cancel upload

Expected behavior

Uploading... message disappears and the file uploading is canceled.

Actual behavior

Uploading... doesn't disappear. The file gets uploaded and the message(Uploading...) disappears.

Server configuration

Operating system: Ubuntu 18.04.2

Web server: Apache/2.4.29 (Ubuntu)

Database: mysql Ver 14.14 Distrib 5.7.25, for Linux (x86_64)

PHP version: PHP 7.2.17-1+ubuntu18.04.1+deb.sury.org+3 (cli) (built: Apr 10 2019 10:50:57) ( NTS )

ownCloud version: (see ownCloud admin page) ownCloud 10.2.0 RC1 (stable)

Updated from an older ownCloud or fresh install: Fresh install

Where did you install ownCloud from: Tarball

Are you using external storage, if yes which one: local/sftp

Are you using encryption: no

Are you using an external user-backend, if yes which one: No

Client configuration

Browser: Firefox 66.0.3 (64-bit)/ Chrome 73

Operating system: Ubuntu 18.04.2

@skshetry skshetry changed the title Uploading files_drop doesnot work if canceled after waiting for few seconds Uploading files_drop does not work if canceled after waiting for few seconds Apr 24, 2019
@individual-it individual-it added this to the QA milestone Apr 24, 2019
@skshetry skshetry changed the title Uploading files_drop does not work if canceled after waiting for few seconds File upload on files_drop does not abort if tried to cancel after waiting for few seconds Apr 24, 2019
@PVince81
Copy link
Contributor

@skshetry can you try on the release-10.2.0 branch ?

I suspect this could be related to #35059 where some initialization code did not work for progress bar. This fix might solve the issue here as well.

@skshetry
Copy link
Member Author

skshetry commented Apr 29, 2019

@PVince81, looks like the bug was when we try to cancel upload after already canceling upload once in the first try.
So, if you:

  1. Create a files_drop link.
  2. Open the share link.
  3. Try to upload a file and then cancel it.
  4. Try to again re-upload and cancel it.

Then, the file upload does not cancel. It works if we reload the tab and retry. Could reproduce in v10.2.0RC1 as well as release-1.2.0.

@PVince81
Copy link
Contributor

ok thanks, so it's a different bug

@PVince81
Copy link
Contributor

@sharidas can you debug this ?

@sharidas
Copy link
Contributor

sharidas commented Apr 30, 2019

While looking into the issue, its been found that when the page is refreshed manually the Upload cancelled appears when the upload is cancelled every time. When the page is not refreshed the Upload cancelled appears for the first time when the upload is cancelled and later on it doesn't appears.

The this.jqXHR here https://github.com/owncloud/core/blob/master/apps/files/js/jquery.fileupload.js#L679 is undefined when the upload of file is done second time and further.

Will add more findings here.

@sharidas
Copy link
Contributor

sharidas commented Apr 30, 2019

Created PR
master branch : #35130
release-10.2.0 branch: #35134

@lock lock bot locked as resolved and limited conversation to collaborators May 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants