274224 2024-05-15 12:58:55 -0700 REGRESSION (276904@main): Breaks https://www.tax.ny.gov/online/. 2025-02-04 09:06:11 -0800 1 1 1 Unclassified WebKit JavaScriptCore WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=271946 https://bugs.webkit.org/show_bug.cgi?id=272082 https://bugs.webkit.org/show_bug.cgi?id=274225 BrowserCompat, InRadar P2 Normal --- 1 mark.lam yijia_huang karlcow webkit-bug-importer oldest_to_newest 2035602 0 mark.lam 2024-05-15 12:58:55 -0700 Steps to reproduce: 1. Navigate to https://www.tax.ny.gov/online/ 2. Click Log in 3. Observe "Network connection was lost" error message or blank white page. This reproduces in MiniBrowser with 276904@main or later. 2035603 1 mark.lam 2024-05-15 12:59:32 -0700 This is caused by https://bugs.webkit.org/show_bug.cgi?id=271946. 2035604 2 webkit-bug-importer 2024-05-15 12:59:48 -0700 <rdar://problem/128149232> 2035605 3 mark.lam 2024-05-15 13:04:50 -0700 We'll have to revert the following: https://commits.webkit.org/276904@main https://commits.webkit.org/277023@main 2035627 4 mark.lam 2024-05-15 13:56:25 -0700 The previous changes were reverted in 278825@main. See https://bugs.webkit.org/show_bug.cgi?id=274225. 2035633 5 mark.lam 2024-05-15 14:01:47 -0700 Honestly, I'm not sure if this is actually a bug in this PR. From my debugging, if I make the getter not prepend "get " to its name, the issue stops reproducing. Meanwhile, Chrome and Firefox all do not implement the spec in https://tc39.es/ecma262/#sec-function.prototype.tostring, and hence, do appear to hit this issue. For example, on Firefox, ``` String(Object.getOwnPropertyDescriptor((function() { "use strict"; }).__proto__, "caller").get) ``` ... yields: ``` "function caller() { [native code] }" ``` Maybe someone needs to tell https://www.tax.ny.gov/online to fix their site, or we need more consensus support from Chrome and Firefox in order to land this. 2037247 6 karlcow 2024-05-22 22:39:48 -0700 Mark, I do not have login access to https://www.tax.ny.gov/online Could you identify which part of the code triggers the bug? Testing with: Safari Technology Preview 191 20619.1.6.3 Firefox Nightly 128.0a1 12824.5.19 Google Chrome Canary 127.0.6495.0 6495.0 So this: String(Object.getOwnPropertyDescriptor((function() { "use strict"; }).__proto__, "caller").get) Safari returns "function caller() { [native code] }" Firefox returns "function caller() { [native code] }" Chrome returns 'function () { [native code] }' 2046735 7 mark.lam 2024-07-17 08:31:49 -0700 (In reply to Karl Dubost from comment #6) > Mark, > > I do not have login access to https://www.tax.ny.gov/online > Could you identify which part of the code triggers the bug? I do not have login access to https://www.tax.ny.gov/online either, but login access is not required. Before 276904@main, trying to login will get some activity on https://www.tax.ny.gov/online. After 276904@main, trying to login gives you a "blank white page". The only change in 276904@main is to rename a getter function: function input() { ... should be: function get input() { ... This is in accordance to the ECMA262 spec. However, after that change in 276904@main (which makes us spec compliant), https://www.tax.ny.gov/online starts showing the "blank white page" on login attempts. What we need to find out here is why that website does not like the spec compliant behavior. 2046825 8 karlcow 2024-07-17 15:17:01 -0700 Given that Firefox returns the same thing than Safari, I wonder if it fails in Firefox too. So on STP Release 198 (Safari 18.0, WebKit 20619.1.20.7) 1. Go to https://www.tax.ny.gov/online/ 2. Click on Login button Result: The site opens a new tab with https://my.ny.gov/LoginV4/login.xhtml?APP=nyappdtf No blank page at the end. (it just takes time 1s to 3s while the page is blank) Same behavior and delay on * Chrome Canary Version 128.0.6598.0 (Official Build) canary (arm64) * Firefox Nightly 128.0 (64 bits) * Working also on a recent internal version of Safari Mark, does it still happen for you? which version? 2046872 9 karlcow 2024-07-17 17:51:14 -0700 comment #8 So my bad, I had forgotten that the change had been reverted so it's normal this is working. 2046873 10 karlcow 2024-07-17 17:51:23 -0700 comment #8 So my bad, I had forgotten that the change had been reverted so it's normal this is working. 2046947 11 karlcow 2024-07-18 00:32:42 -0700 I'm confused with this bug. Object.getOwnPropertyDescriptor(Document.prototype, 'title').get.toString() Chrome: 'function get title() { [native code] }' Firefox: "function title() { [native code] }" Safari: "function title() { [native code] }" In the current state. So Chrome does return the `get` and it is working with the https://www.tax.ny.gov/online/ Now, I also tested by setting Chrome with the UA string for Safari. 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/18.0 Safari/605.1.15' And this is still working. if I set Safari with the Chrome UA Then it fails. So I suspect there is some user agent sniffing going on. which adjusts to the expected string, but I'm not sure where in the code. That also means we can't fix it except by doing a Quirk. Proposal: We re-land the code with set/get and a quirk in the same patch. which removes the set/get just for this site. And we start an outreach effort to make them change. 2047091 12 yijia_huang 2024-07-18 14:24:37 -0700 Pull request: https://github.com/WebKit/WebKit/pull/30980 2047126 13 karlcow 2024-07-18 18:34:28 -0700 We are also in the process of contacting them through radar 132049502 And hopefully we will not have to implement a Quirk. 2056611 14 yijia_huang 2024-08-30 09:55:15 -0700 Pull request: https://github.com/WebKit/WebKit/pull/32950 2089358 15 yijia_huang 2025-01-23 14:51:42 -0800 Pull request: https://github.com/WebKit/WebKit/pull/39468 2091832 16 ews-feeder 2025-02-04 09:06:09 -0800 Committed 289786@main (e4a6a6964d55): <https://commits.webkit.org/289786@main> Reviewed commits have been landed. Closing PR #39468 and removing active labels.