Opened on 03/19/2019 at 01:06:42 PM
Closed on 03/20/2019 at 01:17:23 PM
Last modified on 03/27/2019 at 03:30:27 PM
#7382 closed change (duplicate)
Replace XMLHttpRequest with Fetch
Reported by: | mjethani | Assignee: | mjethani |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Core | Keywords: | |
Cc: | manifestv3 | Blocked By: | |
Blocking: | #5702 | Platform: | Unknown / Cross platform |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
https://gitlab.com/eyeo/adblockplus/adblockpluscore/merge_requests/37 |
Description (last modified by mjethani)
Background
The Downloader and Synchronizer objects use XMLHttpRequest to make requests. This API is no longer going to be available with the Manifest V3 changes. Its usage should be replaced with the Fetch API.
What to change
See patch.
Attachments (0)
Change History (10)
comment:1 Changed on 03/19/2019 at 01:27:19 PM by mjethani
- Component changed from Unknown to Core
- Description modified (diff)
- Priority changed from Unknown to P2
comment:2 Changed on 03/19/2019 at 01:39:40 PM by mjethani
- Summary changed from Replace XMLHttpRequest with fetch to Replace XMLHttpRequest with Fetch
comment:3 Changed on 03/19/2019 at 05:35:22 PM by mjethani
- Owner set to mjethani
comment:4 Changed on 03/20/2019 at 12:47:51 AM by sebastian
- Cc manifestv3 added
comment:5 Changed on 03/20/2019 at 09:17:28 AM by mjethani
comment:6 Changed on 03/20/2019 at 09:17:44 AM by mjethani
- Status changed from new to reviewing
comment:7 Changed on 03/20/2019 at 01:15:38 PM by kzar
- Blocking 7381 removed
comment:8 Changed on 03/20/2019 at 01:17:23 PM by kzar
- Resolution set to duplicate
- Status changed from reviewing to closed
Duplicate of issue 7381 as discussed there.
comment:9 follow-up: ↓ 10 Changed on 03/27/2019 at 03:28:24 PM by mjethani
We also need to add a test for when the subscription URL is invalid.
comment:10 in reply to: ↑ 9 Changed on 03/27/2019 at 03:30:27 PM by mjethani
Replying to mjethani:
We also need to add a test for when the subscription URL is invalid.
Sorry, wrong ticket. ticket:7392#comment:3
For reference, this now also become relevant for manifest V3, as background pages won't have XMLHttpRequest anymore but only fetch() (just like Service Workers).