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

Quota bypassed when uploading with chunked transfer encoding #7993

Closed
ylangisc opened this issue Jan 22, 2018 · 13 comments
Closed

Quota bypassed when uploading with chunked transfer encoding #7993

ylangisc opened this issue Jan 22, 2018 · 13 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: filesystem needs info stale Ticket or PR with no recent activity

Comments

@ylangisc
Copy link

When uploading a file with chunked transfer encoding (https://en.wikipedia.org/wiki/Chunked_transfer_encoding) the quota check is bypassed and you can upload as many bytes over the quota as you want .

Steps to reproduce

  1. Create an account with a low quota
  2. Upload a file that would hit the quota with chunked transfer encoding, i.e. using Mountain Duck which always use this kind of encoding
  3. File is uploaded successfully

Expected behaviour

Server should reply with a 507 response

Actual behaviour

File can be uploaded

Server configuration

Operating system:
Ubuntu 16

Web server:
nginx 1.10.3

Database:
MariaDB

PHP version:
7.0

Nextcloud version: (see Nextcloud admin page)
12.0.4

Updated from an older Nextcloud/ownCloud or fresh install:
Fresh

Where did you install Nextcloud from:
Official download

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

Are you using encryption: yes/no
No

Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...
No

@MorrisJobke
Copy link
Member

cc @icewind1991 @danxuliu

@MorrisJobke
Copy link
Member

@rullzer Mind to test this one?

@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jul 1, 2018
@MorrisJobke MorrisJobke removed this from the Nextcloud 14 milestone Aug 24, 2018
@skjnldsv skjnldsv added the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Jun 12, 2019
@ghost ghost removed the stale Ticket or PR with no recent activity label Jun 12, 2019
@skjnldsv
Copy link
Member

Still valid?

@ghost
Copy link

ghost commented Apr 9, 2020

This issue has been automatically marked as stale because it has not had recent activity and seems to be missing some essential information. It will be closed if no further activity occurs. Thank you for your contributions.

@ghost ghost added the stale Ticket or PR with no recent activity label Apr 9, 2020
@dkocher
Copy link

dkocher commented Apr 9, 2020

Let us know what information is missing to reproduce.

@ghost ghost removed the stale Ticket or PR with no recent activity label Apr 9, 2020
@skjnldsv
Copy link
Member

skjnldsv commented Apr 9, 2020

@dkocher you're still able to reproduce this issue on latest nextcloud stable ?

@ghost
Copy link

ghost commented May 9, 2020

This issue has been automatically marked as stale because it has not had recent activity and seems to be missing some essential information. It will be closed if no further activity occurs. Thank you for your contributions.

@ghost ghost added the stale Ticket or PR with no recent activity label May 9, 2020
@ghost ghost closed this as completed May 23, 2020
@dkocher
Copy link

dkocher commented May 23, 2020

@skjnldsv There is no reason to believe this has been fixed in a later version.

@dkocher
Copy link

dkocher commented May 23, 2020

@nextcloud-bot Please reopen this issue.

@dkocher
Copy link

dkocher commented May 23, 2020

@skjnldsv Auto closing reported bugs with steps to reproduce provided is not gentle and will discourage future bug reports.

@grote
Copy link
Member

grote commented Apr 29, 2024

curl command for op step 2:

curl -H 'Transfer-Encoding: chunked' -H 'Content-Type: image/png' --compressed -v --user "$NEXTCLOUD_USER:$NEXTCLOUD_PASS" -T test.png 'https://cloud.myserver.de/remote.php/webdav/test.png

See that test.png get properly created and can be huge, bypassing quota check.

@grote grote reopened this Apr 29, 2024
@grote grote removed the stale Ticket or PR with no recent activity label Apr 29, 2024
@grote
Copy link
Member

grote commented May 9, 2024

Note that this problem my depend on the server configuration. Nextcloud on a Hetzner Storage Box seems to properly return a 507 error. Or this may have been fixed in a more recent version of Nextcloud.

grote added a commit to grote/seedvault that referenced this issue May 9, 2024
Nextcloud has a bug that lets us write chunked transfers over quota:
nextcloud/server#7993

However, when we upload small files, we can get the proper 507 response and thus detect out of space situations and warn the user about them.
chirayudesai pushed a commit to seedvault-app/seedvault that referenced this issue May 14, 2024
Nextcloud has a bug that lets us write chunked transfers over quota:
nextcloud/server#7993

However, when we upload small files, we can get the proper 507 response and thus detect out of space situations and warn the user about them.
@nextcloud-command
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity and seems to be missing some essential information. It will be closed if no further activity occurs. Thank you for your contributions.

@nextcloud-command nextcloud-command added the stale Ticket or PR with no recent activity label Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: filesystem needs info stale Ticket or PR with no recent activity
Projects
None yet
Development

No branches or pull requests

7 participants