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

Implementing usage of forbidden_chars for file/folder uploads via WebDAV #43603

Closed
nc-fkl opened this issue Feb 15, 2024 · 0 comments · Fixed by #43775
Closed

Implementing usage of forbidden_chars for file/folder uploads via WebDAV #43603

nc-fkl opened this issue Feb 15, 2024 · 0 comments · Fixed by #43775
Assignees
Milestone

Comments

@nc-fkl
Copy link
Contributor

nc-fkl commented Feb 15, 2024

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.

Feature description

Hi Nextcloud Team,

there is a forbidden_characters=[....] feature in the WebUI that disallows the usage for renaming/uploading/moving of files & folders with specific characters that are forbidden.
Could we also implement this for WebDAV so that it is in sync with the frontend?

Thank you for your effort! :)

Describe alternatives you've considered
Updating the documentation that is clear that forbidden_chars is only used in the WebUI and not respected by Desktop Client / WebDav. ( https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/config_sample_php_parameters.html#forbidden-chars )

Additional context

Desktop Client:
nextcloud/desktop#6447

WebUI:
#40585
#38614
#43423

Screenshot Frontend:
+new_folder (6)

@nc-fkl nc-fkl added enhancement feature: dav 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Feb 15, 2024
@susnux susnux added 3. to review Waiting for reviews and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Feb 23, 2024
@susnux susnux self-assigned this Feb 23, 2024
@susnux susnux added this to the Nextcloud 29 milestone Feb 23, 2024
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.

2 participants