Skip to content

Commit 41ced43

Browse files
committed
Improve urlskip implementation
Support multiple instances of regex-based steps. Related issue: uBlockOrigin/uBlock-issues#3914
1 parent 74b838c commit 41ced43

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

src/js/urlskip.js

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
*
7373
* */
7474

75-
export function urlSkip(url, blocked, steps, directive = {}) {
75+
export function urlSkip(url, blocked, steps) {
7676
try {
7777
let redirectBlocked = false;
7878
let urlout = url;
@@ -132,10 +132,7 @@ export function urlSkip(url, blocked, steps, directive = {}) {
132132
}
133133
// Regex extraction from first capture group
134134
if ( c0 === 0x2F ) { // /
135-
const re = directive.cache ?? new RegExp(step.slice(1, -1));
136-
if ( directive.cache === null ) {
137-
directive.cache = re;
138-
}
135+
const re = new RegExp(step.slice(1, -1));
139136
const match = re.exec(urlin);
140137
if ( match === null ) { return; }
141138
if ( match.length <= 1 ) { return; }

0 commit comments

Comments
 (0)