Opened 3 years ago
Last modified 2 months ago
#5182 new change
Fix IsConnectionAllowed
Reported by: | sergz | Assignee: | |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Libadblockplus | Keywords: | |
Cc: | fhd | Blocked By: | #5066 |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | no | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
https://codereview.adblockplus.org/29424786/ |
Description (last modified by sergz)
Background
IsConnectionAllowed had been implemented in a hurry and we had cut some corners, now it's time to fix it.
What to change
Remove it from implementation of XMLHttpRequest in compat.js
However it will be possible only after implementing it in adblockpluscore as #5066.
Change History (9)
comment:1 Changed 3 years ago by sergz
- Blocked By 5198 added
comment:2 Changed 3 years ago by sergz
- Review URL(s) modified (diff)
comment:3 Changed 3 years ago by abpbot
A commit referencing this issue has landed:
Issue 5182 - fix IsConnectionAllowed
comment:4 Changed 3 years ago by sergz
- Blocking 5180 removed
- Description modified (diff)
comment:5 Changed 3 years ago by sergz
- Owner set to sergz
I am working on the part related to #4688, namely removing of LazyWebRequest. It will also cause the change to make IsConnectionAllowed asynchronous. Meanwhile it will be also renamed because the same line will be changed anyway as a part this work.
comment:6 Changed 3 years ago by sergz
- Review URL(s) modified (diff)
comment:7 Changed 3 years ago by abpbot
A commit referencing this issue has landed:
Issue 5182, 4688 - improve IsSubscriptionDowloadAllowedCallback and corresponding tests
comment:8 Changed 3 years ago by sergz
- Blocked By 5179, 5198 removed
- Blocking 3595 removed
- Description modified (diff)
- Owner sergz deleted
- Summary changed from Fix IsAllowedConnection to Fix IsConnectionAllowed
Remove from the issue already landed parts.
I have remove it from JsEngine and from WebRequestThread and made the execution of it asynchronous in FilterEngine, though it still in a detached thread (added blocked by #5198).
The codereview will be after landing of https://codereview.adblockplus.org/29424663/, because otherwise tests don't work anymore, and after https://codereview.adblockplus.org/29424659/ because it uses that code.