1833085 - File downloaded failed due to file name too long

Status: RESOLVED WORKSFORME

(Fenix :: Downloads, defect, P3)

Description

[Mira Lobontiu (Android QA)]

Attachment: download.mp4

Steps to reproduce

1. Go to https://jsfiddle.net/5yzr0gnv/1/show.
2. Tap on "Run this fiddle" play option.
3. Tap on the "Start download" button.
4. Wait for the file to be downloaded, then go to the three-dot menu - Downloads.
5. Observe if the .cvs file is displayed in Downloads.

Expected behavior

The .csv file is displayed.

Actual behavior

1. The .csv file is displayed.
2. The download fails, informing the user to "Try again".

Device information

• Firefox version: Beta 114.0b4, Nightly 115.0a1 from 5/14, RC 113.0
• NOT reproducible on Chrome with Google Pixel 6 device, but reproducible with all the other devices
• Android device: Google Pixel (Android 13) - expected behavior 1
• Android devices: Sony Xperia (Android 6.0.1), Samsung Galaxy Note 8 (Android 9), and LG G7 fit (Android 8.1.0) - for the expected behavior 2

Any additional information?

• I've downloaded other .cvs files, and these are displayed in the Fenix Downloads page.

Comment 1

[jackyzy823]

Error Log

06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService: Unable to complete download for 898e8239-5c1d-4916-85ae-c4dcb0e7b86e marked as FAILED
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService: java.io.FileNotFoundException: /storage/emulated/0/Download/Deep Purple have had several line-up changes and an eight-year hiatus (1976–1984). The first four line-ups, which constituted the band's original 1968–1976 run, are officially indicated as Mark I (1968–1969), Mark II (1969–1973), Mark III (1973–1975) and Mark IV (1975–1976).[12][13] Mark II was the most commercially successful line-up, with Ian Gillan (vocals) and Roger Glover (bass) joining Jon Lord (keyboards), Ian Paice (drums) and Ritchie Blackmore (guitar), the latter three having been founding members of Mark I with Rod Evans (vocals) and Nick Simper (bass). Mark III saw David Coverdale (vocals) and Glenn Hughes (bass and vocals) replace Gillan and Glover, while Mark IV featured Tommy Bolin (guitar) replacing Blackmore. Mark II was revived from 1984–1989 and again from 1992–1993, with Joe Lynn Turner (vocals) replacing Gillan in the intervening 1989–1992 period. Mark II definitively ended in 1993, when Blackmore left Deep Purple for the second and final time. He was replaced temporarily by Joe Satriani (guitar) and then permanently by Steve Morse (guitar). In 2002 Don Airey (keyboards) replaced Lord, which saw Deep Purple settle into its longest running line-up, unchanged for the next twenty years, until Morse announced his departure from the band in 2022. His place was taken by Simon McBride (guitar). Ian Paice, Roger Glover, Ian Gillan, Don Airey and Simon McBride comprise the current line-up of Deep Purple. Deep Purple have had several line-up changes and an eight-year hiatus (1976–1984). The first four line-ups, which constituted the band's original 1968–1976 run, are officially indicated as Mark I (1968–1969), Mark II (1969–1973), Mark III (1973–1975) and Mark IV (1975–1976).[12][13] Mark II was the most commercially successful line-up, with Ian Gillan (vocals) and Roger Glover (bass) joining Jon Lord (keyboards), Ian Paice (drums) and Ritchie Blackmore (guitar), the latter three having been founding members of Mark I with Rod Evans (vocals) and Nick Simper (bass). Mark III saw David Coverdale (vocals) and Glenn Hughes (bass and vocals) replace Gillan and Glover, while Mark IV featured Tommy Bolin (guitar) replacing Blackmore. Mark II was revived from 1984–1989 and again from 1992–1993, with Joe Lynn Turner (vocals) replacing Gillan in the intervening 1989–1992 period. Mark II definitively ended in 1993, when Blackmore left Deep Purple for the second and final time. He was replaced temporarily by Joe Satriani (guitar) and then permanently by Steve Morse (guitar). In 2002 Don Airey (keyboards) replaced Lord, which saw Deep Purple settle into its longest running line-up, unchanged for the next twenty years, until Morse announced his departure from the band in 2022. His place was taken by Simon McBride (guitar). Ian Paice, Roger Glover, Ian Gillan, Don Airey and Simon McBride comprise the current line-up of Deep Purple. .csv (File name too long)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at java.io.FileOutputStream.open(Native Method)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at java.io.FileOutputStream.<init>(FileOutputStream.java:221)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at mozilla.components.feature.downloads.AbstractFetchDownloadService.useFileStreamLegacy$feature_downloads_debug(AbstractFetchDownloadService.kt:945)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at mozilla.components.feature.downloads.AbstractFetchDownloadService.useFileStream$feature_downloads_debug(AbstractFetchDownloadService.kt:863)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at mozilla.components.feature.downloads.AbstractFetchDownloadService$performDownload$1.invoke(AbstractFetchDownloadService.kt:733)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at mozilla.components.feature.downloads.AbstractFetchDownloadService$performDownload$1.invoke(AbstractFetchDownloadService.kt:728)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at mozilla.components.concept.fetch.Response$Body.useStream(Response.kt:79)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at mozilla.components.feature.downloads.AbstractFetchDownloadService.performDownload$feature_downloads_debug(AbstractFetchDownloadService.kt:728)
06-15 23:14:12.256  9248 15493 E AbstractFetchDownloadService:  at mozilla.components.feature.downloads.AbstractFetchDownloa

We should limit the max length of filename.

Comment 2

[Lorand Janos]

Reproducible on the latest Beta (122.0b5) and Nightly (123.0a1 from 2024-01-04) builds.

Device used: Samsung Galaxy S23 Ultra (Android 14).

Comment 3

[Nicholas Poon [:Nick]]

Hi Mirabela, thanks for filing this bug. Would you mind providing another download link because https://jsfiddle.net/5yzr0gnv/1/show is resulting in a 404 error?

Comment 4

[Mira Lobontiu (Android QA)]

I'll close this ticket as WORKSFORME.
I downloaded several files, renamed them with long names, and uploaded them in my Drive. I've redownloaded them and all the files were correctly downloaded and could be accessed from the Downloads folder.
Tested on the Samsung Galaxy Note 8 (Android 9) device, on the latest Fenix Nightly 129.0a1 from 7/2.