Skip to content

[iOS] - Download Filename Incorrect #45433

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

Closed
3 of 5 tasks
Brandon-T opened this issue Apr 14, 2025 · 2 comments · Fixed by brave/brave-core#28656
Closed
3 of 5 tasks

[iOS] - Download Filename Incorrect #45433

Brandon-T opened this issue Apr 14, 2025 · 2 comments · Fixed by brave/brave-core#28656
Assignees

Comments

@Brandon-T
Copy link

Description

Steps to reproduce

  1. https://hackerone.com/reports/3091874

Actual result

  • Downloads not working

Expected result

Filename should show correctly

Reproduces how often

Easily reproduced

Brave version

  • 1.77.x+

Device/iOS version

  • Any

Affected browser versions

  • latest AppStore
  • latest TestFlight
  • previous TestFlight

Reproducibility

  • with Brave Shields disabled
  • in the latest version of mobile Safari

Miscellaneous information

No response

@kjozwiak
Copy link
Member

The above requires 1.78.85 or higher for 1.78.x verification 👍

@hffvld hffvld added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label May 1, 2025
@hffvld
Copy link
Contributor

hffvld commented May 1, 2025

Verified on iPhone 14 using version(s):

Device/OS: iPhone 14 / iOS 17.7.2
Brave build: 1.78 (93)
BraveCore: 1.78.93 (136.0.7103.60)

STEPS:

  1. Follow the STR/TP from https://hackerone.com/reports/3091874
  2. Verify

ACTUAL RESULTS:

  • Verified that downloading a file works as expected.
  • Verified that the correct file extension is shown when downloading a file.

1 2 3 4
1 2 3 4
2025-05-01_16-54-17.mp4

@hffvld hffvld added QA Pass - iPhone and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels May 1, 2025
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.

5 participants