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]: SQL errors when inserting data (Data too long for column 'name') #42551

Closed
5 of 8 tasks
Tracked by #3888
nc-fkl opened this issue Jan 2, 2024 · 3 comments · Fixed by #42598
Closed
5 of 8 tasks
Tracked by #3888

[Bug]: SQL errors when inserting data (Data too long for column 'name') #42551

nc-fkl opened this issue Jan 2, 2024 · 3 comments · Fixed by #42598

Comments

@nc-fkl
Copy link
Contributor

nc-fkl commented Jan 2, 2024

⚠️ This issue respects the following points: ⚠️

Bug description

An user try to upload file with a big file name, but the SQL column size is too small.
The frontend should already decline the upload of that file.

Steps to reproduce

  1. Select a file with a real long filename ( for example: remote.php/webdav/123/abc-defg-hijklmn/img/qr/SnhJUDAwdDU0WHNEWEloYlZNV2RLTnJEdk9xQUt3Z3NEcEZNUk9SL0V2NVFLamJ1TzdJT2dNMzVrSm5IcGVGWFdXUzI3aDZ1QU84S0NVZno5UTZheUdXZExqenkzMTYrd2xYZWJTczlnSEpqdENXL0hqcGFMVTZNOW5NR3ZlM1poK3ZJQUczYwdesgrdhfghnvcbcxfsefsqzedddcferzfdsdxvfseffdfgfgfgdgggfvcvxdf%3D%3D.jpg )
  2. Upload it

Expected behavior

Frontend should decline the upload wit a info/error popup/message

Installation method

Other Community project

Nextcloud Server version

26

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?

None

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

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@nc-fkl nc-fkl added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jan 2, 2024
@sorbaugh
Copy link
Contributor

sorbaugh commented Jan 3, 2024

@skjnldsv do you know by any chance if this this the case in the current version, and if not, is there already a pr somewhere we can backport?

@szaimen
Copy link
Contributor

szaimen commented Jan 3, 2024

cc @nextcloud/files

@skjnldsv
Copy link
Member

skjnldsv commented Jan 5, 2024

Fixed on master already with #41346, 28 backport in #42591

and additional improvements with #42598

@skjnldsv skjnldsv self-assigned this Jan 5, 2024
@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jan 5, 2024
@skjnldsv skjnldsv added this to the Nextcloud 29 milestone Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants