Bug 241322 – Mobile Safari desktop mode user agent reports a different macOS version than Safari on Mac

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Nitin Seshadri]

When Mobile Safari is set to request the desktop website in iOS and iPadOS, it reports a user agent string with the macOS version as 10.15.6:

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.5 Safari/605.1.15

On the other hand, Safari on Mac always reports the macOS version as 10.15.7:

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.5 Safari/605.1.15

This inconsistency should be fixed (i.e. Mobile Safari should report 10.15.7 in desktop mode) because it could make it easier for a website to detect whether the visitor is using Mac Safari or is using Mobile Safari, defeating the purpose of desktop mode.

[Nitin Seshadri]

Created attachment 460042 [details]
Patch

[Sam Sneddon [:gsnedders]]

This has repeatedly bugged me (after all, it provides another way to sniff for iOS v. macOS), but I've never taken the small amount of time to actually fix this. (Thanks Nitin!)

[Alexey Proskuryakov]

Thank you for the patch!

Would you be willing to re-submit it with a commit message? It cannot be landed without one. `webkit-patch upload` would include the commit message, and you can confirm that its format matches existing landed commits.

It looks like https://webkit.org/contributing-code/ is currently out of date, we need to fix it ASAP.

[Nitin Seshadri]

Created attachment 460050 [details]
Patch

[Nitin Seshadri]

(In reply to Alexey Proskuryakov from comment #3)
> Would you be willing to re-submit it with a commit message? It cannot be
> landed without one. `webkit-patch upload` would include the commit message,
> and you can confirm that its format matches existing landed commits.

done!

[Alexey Proskuryakov]

Sorry, it's not quite it yet (needs a "Reviewed by NOBODY (OOPS!)" line, otherwise tooling that replaces it with an actual reviewer name won't work).

I think that if you reset, run `git-webkit setup`, and re-commit, the correct template should be generated.

[Nitin Seshadri]

Created attachment 460054 [details]
Patch

[Nitin Seshadri]

(In reply to Alexey Proskuryakov from comment #6)
> Sorry, it's not quite it yet (needs a "Reviewed by NOBODY (OOPS!)" line,
> otherwise tooling that replaces it with an actual reviewer name won't work).
> 
> I think that if you reset, run `git-webkit setup`, and re-commit, the
> correct template should be generated.
Should be fixed now.

[Alexey Proskuryakov]

Comment on attachment 460054 [details]
Patch

Let's see what happens!

[EWS]

Committed r295305 (251347@main): <https://commits.webkit.org/251347@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 460054 [details].

[Radar WebKit Bug Importer]

<rdar://problem/94474123>