Skip to content

Conversation

@FearLeSS-21
Copy link
Contributor

No description provided.

@CostaGCore CostaGCore requested a review from oborodin January 16, 2026 12:26
@CostaGCore CostaGCore self-assigned this Jan 16, 2026
@CostaGCore CostaGCore requested review from CostaGCore and removed request for oborodin January 16, 2026 12:27
Copy link

@CostaGCore CostaGCore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved


async testDatasource(): Promise<any> {
try {
// TODO: Use the fetch function instead datasourceRequest

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you committing this TODO? Where you supposed to convert it to use fetch?

);
}

// TODO: Use the fetch function instead datasourceRequest

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you committing this TODO? Where you supposed to convert it to use fetch?

@CostaGCore CostaGCore merged commit b6d4022 into G-Core:master Jan 16, 2026
1 of 2 checks passed
version "10.5.0"
resolved "https://registry.yarnpkg.com/glob/-/glob-10.5.0.tgz#8ec0355919cd3338c28428a23d4f24ecc5fe738c"
integrity sha512-DfXN8DfhJ7NH3Oe7cFmu3NCu1wKbkReJ8TorzSAFbSKrlNaQSKfIzqYqVY8zlbs2NLBbWpRiU52GX2PbaBVNkg==
glob@^10.2.7:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The glob 10.2.7 is marked by Dependabot alerts "Command injection via -c/--cmd executes matches with shell:true"

https://github.com/G-Core/cdn-stats-datasource-plugin/security/dependabot/103

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@FearLeSS-21 @CostaGCore, please take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants