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

Propagate max chunk size downstream to clients #38138

Open
gymnae opened this issue May 9, 2023 · 0 comments
Open

Propagate max chunk size downstream to clients #38138

gymnae opened this issue May 9, 2023 · 0 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement feature: dav

Comments

@gymnae
Copy link

gymnae commented May 9, 2023

How to use GitHub

  • Please use the 👍 reaction to show that you are interested into the same feature.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Is your feature request related to a problem? Please describe.
If Nextcloud is behind a protective layer, for example Cloudflare, there is the need to set up chunking to enable the upload of files bigger than 100 MB.
Currently, this needs to be done manually in all locations which want to interact with the Nextcloud installation:

  1. mobile clients
  2. Server via OCC command
  3. manual config file edit for Desktop clients

Especially the last part is a dealbreaker, since I cannot ask “simple” users to go and edit a config file stashed away somewhere and add a line, which I need to copy to them.

Describe the solution you'd like
The server should act as a central instance which can dictate the max size of chunks. Clients can manually opt for smaller chunks, but not bigger. The max chunk size is automatically stored in the config of the clients, without the need of them running into a 413 error and reducing the chunk size until it fits.

Describe alternatives you've considered
Hand tune every client installation by ensuring that they edit the config file or setting in the app. But, as pointed out, this solution doesn't scale.

There is at least this PR in the pipeline which could partly address the issue: nextcloud/desktop#4826

Additional context
Add any other context or screenshots about the feature request here.

@gymnae gymnae added 0. Needs triage Pending check for reproducibility or if it fits our roadmap enhancement labels May 9, 2023
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 enhancement feature: dav
Projects
None yet
Development

No branches or pull requests

2 participants