New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: skip `parseESlint` if eslint filepath is a js file #510

Merged

antfu merged 2 commits into unplugin:main from yy4382:eslint-js-parse

Jul 11, 2024

Merged

fix: skip `parseESlint` if eslint filepath is a js file #510

chore: update

Annotations

8 errors and 2 warnings

src/core/ctx.ts#L116

Unescaped source character '{'

src/core/ctx.ts#L116

Unescaped source character '}'

src/core/ctx.ts#L117

Unescaped source character '{'

src/core/ctx.ts#L117

Unescaped source character '}'

src/core/ctx.ts#L127

The quantifier '[^\s'"]+' can exchange characters with '.+?'. Using any string accepted by /:+/, this can be exploited to cause at least polynomial backtracking

src/core/ctx.ts#L127

The quantifier '[^\s'"]+' can exchange characters (:) with '.+?'. This makes the capturing group misleading, because the quantifier will capture fewer characters than its pattern suggests

src/core/ctx.ts#L127

The quantifier '\s*' can exchange characters with '.+?'. Using any string accepted by /[\t\x0b\f \xa0\u1680\u2000-\u200a\u202f\u205f\u3000\ufeff]+/, this can be exploited to cause at least polynomial backtracking

Process completed with exit code 1.

The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

src/core/ctx.ts#L114

The 'm' flag is unnecessary because the pattern does not contain start (^) or end ($) assertions

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: skip `parseESlint` if eslint filepath is a js file #510

fix: skip `parseESlint` if eslint filepath is a js file #510

Annotations

This job failed

Re-running jobs...

fix: skip parseESlint if eslint filepath is a js file #510

chore: update

fix: skip parseESlint if eslint filepath is a js file #510

Annotations

This job failed

Re-running jobs...

fix: skip `parseESlint` if eslint filepath is a js file #510

fix: skip `parseESlint` if eslint filepath is a js file #510