| Summary: | REGRESSION (253839@main): Opening projects broken in Construct 3 web app with Safari TP 163 | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Ashley Gullen <ashley> | ||||
| Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Major | CC: | brandonstewart, cdumez, simon.fraser, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | Safari Technology Preview | ||||||
| Hardware: | Mac (Apple Silicon) | ||||||
| OS: | macOS 13 | ||||||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=253042 https://bugs.webkit.org/show_bug.cgi?id=254021 |
||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 244350 | ||||||
| Attachments: |
|
||||||
|
Description
Ashley Gullen
2023-02-17 04:49:30 PST
*** Bug 252664 has been marked as a duplicate of this bug. *** This issue is not a duplicate of issue 252474! Please unmerge them or you will likely leave a breaking change in Safari! 253433@main: GOOD 254508@main: BAD Will see if I can bisect this. (In reply to Chris Dumez from comment #4) > 253433@main: GOOD > 254508@main: BAD > > Will see if I can bisect this. Regression range: https://commits.webkit.org/compare/253849@main...253835@main Couldn't bisect further. (In reply to Chris Dumez from comment #5) > (In reply to Chris Dumez from comment #4) > > 253433@main: GOOD > > 254508@main: BAD > > > > Will see if I can bisect this. > > Regression range: > https://commits.webkit.org/compare/253849@main...253835@main > > Couldn't bisect further. I may have messed up my bisection. Double checking now... (In reply to Chris Dumez from comment #6) > (In reply to Chris Dumez from comment #5) > > (In reply to Chris Dumez from comment #4) > > > 253433@main: GOOD > > > 254508@main: BAD > > > > > > Will see if I can bisect this. > > > > Regression range: > > https://commits.webkit.org/compare/253849@main...253835@main > > > > Couldn't bisect further. > > I may have messed up my bisection. Double checking now... I think my bisection is correct: 253835@main-1661538933_6bad2b65adc8: GOOD 253849@main-1661559262_cb2c1049f95b: BAD This is a regression from https://github.com/WebKit/WebKit/commit/c5b77d735efefca6e22d8dfbea5aba69daaf3892 Site may be using the stream compression API somehow. Just took a quick look. Disabling Compression Streams does seem to resolve the error. I checked in Chrome, and they do not have an issue, and they have Compression Stream support. I'll debug this in depth tonight, and see exactly where the error is happening in WebKit. Quick fix posted https://github.com/WebKit/WebKit/pull/10768. I'll need to look into that a bit more, but just switched to using the zlib library implementation for now while I investigate. Landed the fix. The site is working properly now based on initial testing. Thanks for the quick fix. FYI I think the use of Compression Streams comes from the popular zip-reading library zip.js (https://gildas-lormeau.github.io/zip.js/) so I suspect there is a much wider compatibility issue at stake. This issue is marked fixed but it still reproduces for me in Safari TP 165. Please can you tell me if the fix is really going to ship in Safari 16.4 so I know if we need to issue an update to work around this? (In reply to Ashley Gullen from comment #14) > This issue is marked fixed but it still reproduces for me in Safari TP 165. > Please can you tell me if the fix is really going to ship in Safari 16.4 so > I know if we need to issue an update to work around this? The fix landed in 260923@main. However, per [1], STP 165 only contains fixes up to 260848@main. [1] https://webkit.org/blog/13932/release-notes-for-safari-technology-preview-165/ |