RESOLVED FIXED 208387
Adopt HTTP Alternative Services Storage
https://bugs.webkit.org/show_bug.cgi?id=208387
Summary Adopt HTTP Alternative Services Storage
jmehta
Reported 2020-02-28 12:53:46 PST
Attachments
Proposed changes for HTTP Alternative Services (15.98 KB, patch)
2020-02-28 12:57 PST, jmehta
no flags
Proposed changes for HTTP Alternative Services (15.99 KB, patch)
2020-02-28 13:58 PST, jmehta
no flags
Patch (31.53 KB, patch)
2020-03-02 16:23 PST, Alex Christensen
no flags
patch (31.53 KB, patch)
2020-03-02 17:06 PST, Alex Christensen
no flags
Patch (35.05 KB, patch)
2020-03-02 23:31 PST, Alex Christensen
no flags
Patch (39.88 KB, patch)
2020-03-03 09:34 PST, Alex Christensen
no flags
jmehta
Comment 1 2020-02-28 12:57:05 PST
Created attachment 392007 [details] Proposed changes for HTTP Alternative Services
jmehta
Comment 2 2020-02-28 13:58:18 PST
Created attachment 392016 [details] Proposed changes for HTTP Alternative Services
Maciej Stachowiak
Comment 3 2020-02-29 14:17:15 PST
Besides I breaking the build, we need to verify that this doesn’t create a potential supercookie like HSTS did, that it’s cleared by clearing website data, etc.
Alex Christensen
Comment 4 2020-02-29 14:19:19 PST
Comment on attachment 392016 [details] Proposed changes for HTTP Alternative Services This needs a new HAVE_ALTERNATIVE_SERVICES macro in Platform.h to not break the build. We will also need SPI on _WKWebsiteDataStoreConfiguration to specify the storage file location.
Alex Christensen
Comment 5 2020-03-02 16:23:27 PST
Alex Christensen
Comment 6 2020-03-02 17:06:07 PST
Alex Christensen
Comment 7 2020-03-02 23:31:23 PST
Alex Christensen
Comment 8 2020-03-03 09:34:55 PST
WebKit Commit Bot
Comment 9 2020-03-03 10:21:22 PST
Comment on attachment 392285 [details] Patch Clearing flags on attachment: 392285 Committed r257785: <https://trac.webkit.org/changeset/257785>
WebKit Commit Bot
Comment 10 2020-03-03 10:21:24 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 11 2020-03-03 10:22:26 PST
Note You need to log in before you can comment on or make changes to this bug.