Add WebKit API to turn off the URL scheme check when linkifying through Data Detectors. This will enable us to block the Mach service com.apple.lsd.open in the WebContent process for all clients.
Created attachment 458587 [details] Patch
<rdar://92026172>
Created attachment 458591 [details] Patch
Comment on attachment 458591 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=458591&action=review > Source/WebKit/ChangeLog:10 > + The URL scheme check is disabled by default for all clients that are not Web browsers. Probably worth a ChangeLog comment explaining why we made this default choice. Come to think of it, why did we make this default choice? I guess our motivating example was Mail clients? Maybe we should enable by default just for apps with the com.apple.developer.mail-client entitlement?
(In reply to Geoffrey Garen from comment #4) > Comment on attachment 458591 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=458591&action=review > > > Source/WebKit/ChangeLog:10 > > + The URL scheme check is disabled by default for all clients that are not Web browsers. > > Probably worth a ChangeLog comment explaining why we made this default > choice. > Will add! > Come to think of it, why did we make this default choice? I guess our > motivating example was Mail clients? Maybe we should enable by default just > for apps with the com.apple.developer.mail-client entitlement? That is a good question. IIRC, we made this choice because we from beforehand knew that com.apple.lsd.open can be blocked for Web browsers, since we already have blocked it there for quite some time without observing any issues. I think we may also have a couple of examples of other apps that are not Mail clients that would need this to be the default behavior. On the other hand, the apps that are not Mail clients could use the new API :) Thanks for reviewing!
Created attachment 459186 [details] Patch
Still holding off landing a bit, since it appears that some builders are not ready yet.
Created attachment 459443 [details] Patch
Created attachment 459499 [details] Patch
Created attachment 459506 [details] Patch
Pull request: https://github.com/WebKit/WebKit/pull/693
Committed r294397 (250691@main): <https://commits.webkit.org/250691@main> Reviewed commits have been landed. Closing PR #693 and removing active labels.