Skip to content

Commit 9350dbe

Browse files
committed
fix: fix redirect condition
1 parent e834ab0 commit 9350dbe

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

src/no-dead-link.js

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -294,17 +294,20 @@ function reporter(context, options = {}) {
294294
? await isAliveLocalFile(uri)
295295
: await memorizedIsAliveURI(uri, method, maxRetryCount);
296296
const { ok, redirected, redirectTo, message } = result;
297-
298-
if (redirected && !ruleOptions.ignoreRedirects) {
297+
// When ignoreRedirects is true, redirected should be ignore
298+
if (redirected && ruleOptions.ignoreRedirects) {
299+
return;
300+
}
301+
if (!ok) {
302+
const lintMessage = `${uri} is dead. (${message})`;
303+
report(node, new RuleError(lintMessage, { index }));
304+
} else if (redirected) {
299305
const lintMessage = `${uri} is redirected to ${redirectTo}. (${message})`;
300306
const fix = fixer.replaceTextRange(
301307
[index, index + uri.length],
302308
redirectTo,
303309
);
304310
report(node, new RuleError(lintMessage, { fix, index }));
305-
} else if (!ok) {
306-
const lintMessage = `${uri} is dead. (${message})`;
307-
report(node, new RuleError(lintMessage, { index }));
308311
}
309312
};
310313

0 commit comments

Comments
 (0)