Bug 237269 - allow-custom-protocols-navigation sandbox flag.
Summary: allow-custom-protocols-navigation sandbox flag.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks: 237605
  Show dependency treegraph
 
Reported: 2022-02-28 04:42 PST by Arthur Sonzogni
Modified: 2022-03-08 07:23 PST (History)
9 users (show)

See Also:


Attachments
Patch (8.37 KB, patch)
2022-03-07 08:52 PST, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arthur Sonzogni 2022-02-28 04:42:22 PST
Feature request: https://github.com/whatwg/html/pull/7654

The freshly minted "allow-custom-protocols-navigation" would navigations toward external applications to continue to work inside a sandboxed iframe. 

This is a follow-up to:
https://bugs.webkit.org/show_bug.cgi?id=231727
which has been implemented in Webkit with quirks for microsoft team. Implementing it would allow removing the quirk, once Microsoft Team start using it.
Comment 1 Radar WebKit Bug Importer 2022-03-07 04:43:14 PST
<rdar://problem/89899736>
Comment 2 Chris Dumez 2022-03-07 08:52:49 PST
Created attachment 453995 [details]
Patch
Comment 3 Geoffrey Garen 2022-03-07 13:34:45 PST
Comment on attachment 453995 [details]
Patch

r=me
Comment 4 EWS 2022-03-07 14:33:58 PST
Committed r290958 (248140@main): <https://commits.webkit.org/248140@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 453995 [details].
Comment 5 Arthur Sonzogni 2022-03-08 01:15:18 PST
Hi Chris,

You landed "allow-custom-protocols-navigation". Great!

This was the initial name, but it changed to:
"allow-top-navigation-to-custom-protocols"

This was requested here:
https://github.com/whatwg/html/pull/7654#issuecomment-1054219407

You probably want to make a correction to your patch.
Comment 6 Chris Dumez 2022-03-08 07:09:37 PST
(In reply to Arthur Sonzogni from comment #5)
> Hi Chris,
> 
> You landed "allow-custom-protocols-navigation". Great!
> 
> This was the initial name, but it changed to:
> "allow-top-navigation-to-custom-protocols"
> 
> This was requested here:
> https://github.com/whatwg/html/pull/7654#issuecomment-1054219407
> 
> You probably want to make a correction to your patch.

Ouch! Thanks for pointing this out. I'll follow-up.
Comment 7 Chris Dumez 2022-03-08 07:23:28 PST
(In reply to Chris Dumez from comment #6)
> (In reply to Arthur Sonzogni from comment #5)
> > Hi Chris,
> > 
> > You landed "allow-custom-protocols-navigation". Great!
> > 
> > This was the initial name, but it changed to:
> > "allow-top-navigation-to-custom-protocols"
> > 
> > This was requested here:
> > https://github.com/whatwg/html/pull/7654#issuecomment-1054219407
> > 
> > You probably want to make a correction to your patch.
> 
> Ouch! Thanks for pointing this out. I'll follow-up.

Following up in Bug 237605.