Skip to content
This repository was archived by the owner on Apr 18, 2026. It is now read-only.

Integrate deps#1878

Merged
risadams merged 2 commits into
masterfrom
integrate-deps
Oct 2, 2022
Merged

Integrate deps#1878
risadams merged 2 commits into
masterfrom
integrate-deps

Conversation

@risadams

@risadams risadams commented Oct 2, 2022

Copy link
Copy Markdown
Contributor

Integrates Sifter and Microplugin dependencies.

github-advanced-security[bot]
github-advanced-security AI found potential problems Oct 2, 2022
View reviewed changes
Comment thread src/contrib/sifter.js
};

var trim = function (str) {
return (str + '').replace(/^\s+|\s+$|/g, '');

Check failure

Code scanning / CodeQL

Polynomial regular expression used on uncontrolled data

This [regular expression](1) that depends on [library input](2) may run slow on strings with many repetitions of ' '. This [regular expression](1) that depends on [library input](3) may run slow on strings with many repetitions of ' '.
@risadams risadams merged commit eb62c8a into master Oct 2, 2022
@risadams risadams deleted the integrate-deps branch October 2, 2022 15:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

hacktoberfest-accepted

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@risadams @github-advanced-security