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

[Bug]: 31.0.0 RC2 File Drop folder fails #50584

Open
5 of 8 tasks
K3das opened this issue Jan 31, 2025 · 2 comments · May be fixed by #50591
Open
5 of 8 tasks

[Bug]: 31.0.0 RC2 File Drop folder fails #50584

K3das opened this issue Jan 31, 2025 · 2 comments · May be fixed by #50591
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 31-feedback bug feature: sharing

Comments

@K3das
Copy link

K3das commented Jan 31, 2025

⚠️ This issue respects the following points: ⚠️

Bug description

When uploading a folder through file drop on 31.0.0 rc2, the WebDav MKCOL request fails with Only PUT is allowed on files drop returned in the HTTP response. No errors are displayed in the UI, but "Unknown error during upload" is displayed when trying to cancel the upload.

Steps to reproduce

  1. Run NC 31 rc2 (docker run --rm -p 8080:80 lscr.io/linuxserver/nextcloud:develop-version-31.0.0rc2)
  2. Create a file drop with default permissions and configuration
  3. Upload a folder using "upload folders"

Expected behavior

Folder uploads

Nextcloud Server version

31

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

Fresh Nextcloud Server install

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

List of activated Apps

Nextcloud Signing status

Nextcloud Logs

Additional info

No response

@K3das K3das added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Jan 31, 2025
@szaimen
Copy link
Contributor

szaimen commented Jan 31, 2025

FYI @susnux

@susnux
Copy link
Contributor

susnux commented Jan 31, 2025

I think we need to split this issue into two:

  1. Its a bug that you see the upload folder option
  2. Add a feature request to support uploading folders ( Allow uploading folders on file-drop ("upload only") shares #15921 )

Uploading folders also did not work on older Nextcloud versions.
The problem why we forbid this is that for files we ensure they are unique when uploading (or we rename them if needed).
For folders the problem is there is currently no logic to restrict uploaders to the folder they created to prevent writing into new folders of other uploaders.

So this should be fixed first before allowing folder upload.

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 31-feedback bug feature: sharing
Projects
None yet
4 participants