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

"An unknown error has occurred" when not enough quota #12588

Closed
nickvergessen opened this issue Nov 22, 2018 · 3 comments · Fixed by #12866
Closed

"An unknown error has occurred" when not enough quota #12588

nickvergessen opened this issue Nov 22, 2018 · 3 comments · Fixed by #12866

Comments

@nickvergessen
Copy link
Member

nickvergessen commented Nov 22, 2018

Happens on master:

  1. Set the quota of a user to 0 or a few bytes
  2. Upload a file as that user
  3. Expected ❓
    } else if (status === 507) {
    // not enough space
    OC.Notification.show(t('files', 'Not enough free space'), {type: 'error'});
    self.cancelUploads();
    or
    freeSpace = $('#free_space').val();
    if (freeSpace >= 0 && selection.totalBytes > freeSpace) {
    data.textStatus = 'notenoughspace';
    data.errorThrown = t('files',
    'Not enough free space, you are uploading {size1} but only {size2} is left', {
    'size1': humanFileSize(selection.totalBytes),
    'size2': humanFileSize($('#free_space').val())
    });
    }
  4. Actually throws 💥
    // HTTP connection problem or other error
    var message = t('files', 'An unknown error has occurred');
    if (upload) {
    var response = upload.getResponse();
    if (response) {
    message = response.message;
    }
    }
    OC.Notification.show(message || data.errorThrown, {type: 'error'});

Works fine in stable14

@skjnldsv can you have a look? You modified the upload stuff in general lately?

@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #2017 (quota), #9960 (Error occurred during the update process), #8618 (error occurs when try to enable "External Storage Support"), #8050 (Quota issues ), and #3958 (An error occured: Bad signature - corrupted encrypted files).

@skjnldsv
Copy link
Member

Ah yes, introduced by a793b10

@rullzer rullzer added this to the Nextcloud 15 milestone Nov 22, 2018
@ChristophWurst
Copy link
Member

@skjnldsv ping? A fix would be awesome ❤️

@skjnldsv skjnldsv self-assigned this Dec 1, 2018
@MorrisJobke MorrisJobke mentioned this issue Dec 5, 2018
12 tasks
rullzer added a commit that referenced this issue Dec 5, 2018
Fixes #12588
Probably needs more fixing for the other cases. But this is the quick
fix I could come up with for now.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
backportbot-nextcloud bot pushed a commit that referenced this issue Dec 6, 2018
Fixes #12588
Probably needs more fixing for the other cases. But this is the quick
fix I could come up with for now.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants