Closed Bug 1916193 Opened 2 years ago Closed 2 years ago

Stop exempting the Windows Store protocol handler from permissions checks in Firefox (was: Don't allow Firefox to automatically open the Windows Store app upon clicking a Windows Store URL)

Categories

(Firefox :: File Handling, defect)

Desktop
Windows
defect

Tracking

()

VERIFIED FIXED
132 Branch
Tracking Status
firefox-esr115 --- wontfix
firefox-esr128 --- verified
firefox130 --- wontfix
firefox131 --- wontfix
firefox132 --- verified
firefox133 --- verified

People

(Reporter: Gijs, Assigned: Gijs)

References

(Regression)

Details

(Keywords: parity-chrome, regression)

Attachments

(2 files)

(Cloned from bug 1913147)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:129.0) Gecko/20100101 Firefox/129.0

Steps to reproduce:

This is related to a bug that was closed without any change to this issue (bug 1567206). <snip>

Open any Windows store page on a Windows version of Firefox, e.g.
https://apps.microsoft.com/detail/9nksqgp7f2nh?hl=en-us&gl=US
Click on the "View in Store" button

Actual results:

The Windows App Store in the operating system opens

Expected results:

Firefox should prompt for permission before opening the Windows store app.


This was previously discussed in bug 1567206. The separate default comes from bug 866065, before we had per-site permissions for protocols. The UX now is better, and we're no longer trying to build for Metro and its single-window modes, so I think it's fine to provide the same security functionality as Chrome, rather than following Edge (which opens without prompting).

Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
See Also: → 1567206
See Also: → 1677753
Blocks: 1677753
See Also: 1677753 →
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/20660a3f6cfa stop always allowing opening the Windows Store, r=dveditz
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch

The patch landed in nightly and beta is affected.
:Gijs, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox131 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(gijskruitbosch+bugs)

I think this is fine to ride the trains. We can evaluate ESR once this has been out on release.

Flags: needinfo?(gijskruitbosch+bugs)
Flags: qe-verify+

I was able to reproduce the issue on Win11x64 using FF build 131.0a1 (2024-08-30).
Verified as fixed on Win11x64 using FF build 133.0a1 and 132.0b2. On Ubuntu 24.04 and Mac 12.6 this button is not present.

(In reply to :Gijs (he/him) from comment #5)

I think this is fine to ride the trains. We can evaluate ESR once this has been out on release.

This has been on release for a couple cycles now. What are your thoughts on ESR?

Flags: needinfo?(gijskruitbosch+bugs)
Attachment #9443575 - Flags: approval-mozilla-esr128?

esr128 Uplift Approval Request

  • User impact if declined: Users lack control over windows store behaviour
  • Code covered by automated testing: no
  • Fix verified in Nightly: yes
  • Needs manual QE test: no
  • Steps to reproduce for manual QE testing: n/a
  • Risk associated with taking this patch: very low
  • Explanation of risk level: this just changes some pref defaults
  • String changes made/needed: No
  • Is Android affected?: no

(In reply to Ryan VanderMeulen [:RyanVM] from comment #7)

(In reply to :Gijs (he/him) from comment #5)

I think this is fine to ride the trains. We can evaluate ESR once this has been out on release.

This has been on release for a couple cycles now. What are your thoughts on ESR?

Sure, why not. :-)

Flags: needinfo?(gijskruitbosch+bugs)
Attachment #9443575 - Flags: approval-mozilla-esr128? → approval-mozilla-esr128+

Verified as fixed on Win11x64 using FF build 128.6.0esr(20241218153108).

Status: RESOLVED → VERIFIED
Flags: qe-verify+
Duplicate of this bug: 1927192
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: