Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

guessFileName should be able to handle generic content types #9526

Closed
Amejia481 opened this issue Jan 27, 2021 · 0 comments
Closed

guessFileName should be able to handle generic content types #9526

Amejia481 opened this issue Jan 27, 2021 · 0 comments
Assignees
@Amejia481
Labels
🐞 bug Something isn't working <download> Component: feature-download
Milestone
73.0.0 🍧

Comments

@Amejia481
Copy link
Contributor

Amejia481 commented Jan 27, 2021 •
edited by data-sync-user
Loading

When we extract a file name from an URL we have to take into consideration if the provided content type is a generic one, if so we should not try to change the file extension.

Related tickets mozilla-mobile/fenix#17282 mozilla-mobile/fenix#17263

┆Issue is synchronized with this Jira Task
@Amejia481 Amejia481 self-assigned this Jan 27, 2021
@Amejia481 Amejia481 added <download> Component: feature-download 🐞 bug Something isn't working labels Jan 27, 2021
@Amejia481 Amejia481 added this to the 73.0.0 milestone Jan 27, 2021
Amejia481 added a commit to Amejia481/android-components that referenced this issue Jan 27, 2021
@Amejia481
Closes mozilla-mobile#9526 guessFileName handle generic content types
7f5e09f
@mergify mergify bot closed this as completed in 1f6341b Jan 28, 2021
@Amejia481 Amejia481 mentioned this issue Jan 29, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Amejia481 Amejia481

Labels
🐞 bug Something isn't working <download> Component: feature-download
Projects
None yet
Milestone
73.0.0 🍧
Development

No branches or pull requests
1 participant
@Amejia481