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

Really long filenames break ingest #5436

Closed
1 task
elynema opened this issue Oct 20, 2023 · 2 comments
Closed
1 task

Really long filenames break ingest #5436

elynema opened this issue Oct 20, 2023 · 2 comments
Assignees
Labels

Comments

@elynema
Copy link
Contributor

elynema commented Oct 20, 2023

Pulled over from local board #174

Description

[Chris]: It appears that really long filenames overflow the encode dashboard database record for the title field: https://app.honeybadger.io/projects/54117/faults/99716515
A DB migration may be needed to modify that column to change the datatype or a truncation of that field in ActiveEncode::EncodeRecord before attempting to save it.

[Chris]: I ran into this when trying to load a batch on avalon-sandbox as part of #5397. avalon-sandbox was using minio for the dropbox and a mysql database when this happened. I temporarily worked around it by increasing the size of title to varchar(1000) and display_title to varchar(750).
This should probably be moved to the core board and handled there.

Done Looks Like

  • A user can ingest a filename with up to 256 characters either by web upload or batch ingest
@elynema
Copy link
Contributor Author

elynema commented Nov 3, 2023

Move this over to mco-staging for testing and consider deploying out to production?

@joncameron
Copy link
Contributor

Tested with filename hereisareallylongfilenamewithnospacesorextracharacterscommathatistosaynothingwhichshouldcreateproblemsinfilenamehandlingotherthanthefacethatitisjustveryverylongandthatisthelongopenparenthesislolcloseparenthesisandshortofit.MOV,
227 total characters and things worked as expected: https://avalon-dev.dlib.indiana.edu/media_objects/mw22v546v.
The encoding dashboard, however, expanded horizontally to fit this, which looks funky but could be changed with truncation of the filename in the table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants