Skip to content
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

Remove test files from the npm package! F-Secure warning #240

Open
Havunen opened this issue Sep 17, 2024 · 2 comments
Open

Remove test files from the npm package! F-Secure warning #240

Havunen opened this issue Sep 17, 2024 · 2 comments
Labels
Status: Accepted It's clear what the subject of the issue is about, and what the resolution should be. Type: Maintenance Updating phrasing or wording to make things clearer or removing ambiguity, without a functionality.

Comments

@Havunen
Copy link

Havunen commented Sep 17, 2024

koajs/body-parsers#89

The dependency chain seems to be following:

"koa-better-body": "^3.3.9",

Loads "koa-body-parsers": "tunnckocore/body-parsers#patch-1"

which seems to be hosted here:

https://github.com/tunnckoCore/body-parsers/tree/patch-1

and it contains a test file shipped in the npm package.

Note: Koa-better-body is targeting koa-body-parser 3.1.0 is that still needed could it be updated to the official v5 ?

@Havunen
Copy link
Author

Havunen commented Sep 17, 2024

https://www.f-secure.com/v-descs/trojan-downloader-js-maljas.shtml

image

Technical details and removal instructions for programs and files detected by F-Secure products.

@tunnckoCore
Copy link
Owner

@Havunen hey there. Sorry for that. I'm not using koa much in the recent years and i have to catch up and update them. There was a reason why i specifically used my patch, have to review the v5.

@tunnckoCore tunnckoCore added Status: Accepted It's clear what the subject of the issue is about, and what the resolution should be. Type: Maintenance Updating phrasing or wording to make things clearer or removing ambiguity, without a functionality. labels Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Accepted It's clear what the subject of the issue is about, and what the resolution should be. Type: Maintenance Updating phrasing or wording to make things clearer or removing ambiguity, without a functionality.
Projects
None yet
Development

No branches or pull requests

2 participants