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

[Snyk] Fix for 3 vulnerabilities #4711

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rubenfiszel
Copy link
Contributor

@rubenfiszel rubenfiszel commented Nov 14, 2024

snyk-top-banner

Snyk has created this PR to fix 3 vulnerabilities in the npm dependencies of this project.

Snyk changed the following file(s):

  • frontend/package.json
  • frontend/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
high severity Prototype Pollution
SNYK-JS-AGGRIDCOMMUNITY-7414157
high severity Prototype Pollution
SNYK-JS-AGGRIDENTERPRISE-7414158
medium severity Uncontrolled Resource Consumption ('Resource Exhaustion')
SNYK-JS-TAR-6476909

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Prototype Pollution
🦉 Uncontrolled Resource Consumption ('Resource Exhaustion')


Important

Upgrade ag-grid-community, ag-grid-enterprise, and pdfjs-dist in package.json to fix security vulnerabilities.

  • Dependencies:
    • Upgrade ag-grid-community from ^31.0.0 to ^31.3.4 in package.json.
    • Upgrade ag-grid-enterprise from ^31.0.0 to ^31.3.4 in package.json.
    • Upgrade pdfjs-dist from ^3.8.162 to ^4.4.168 in package.json.
  • Vulnerabilities Fixed:
    • Fixes Prototype Pollution in ag-grid-community and ag-grid-enterprise.
    • Fixes Uncontrolled Resource Consumption in tar.

This description was created by Ellipsis for 6c59565. It will automatically update as commits are pushed.

dieriba and others added 2 commits November 14, 2024 14:17
* wip: openai proxy and other ai proxy integration

* fixing migration script

* wip: support different ai provider in front, fix proxy openai

* wip: adding frontend ai provider

* updated copilot types

* wip: working on anthropic integration

* done AI proxy front

* adding new type and support for anthropic

* updating gitignore

* adding streaming response

* added streaming prompt

* push lib/gen

* wip: fixing anthropic

* anthropic fully supported

* fix backend missing var error and fully support stream event for anthropic

* remove gen directory

* fixing openapi file

* add support for mistral, and update create workspace components

* remove deref.json

* remove package-json

* openapi

* fix ui enable code

* added utility function for init workspace ai provider

* fix workspace switch bug

* update anthropic property and fixed frontend error

* fix workspace settings

* update error message and fix typo migration file

* chore: update openapi file

* fix dev file

* add .sqlx

* all

* update sqlx

---------

Co-authored-by: dieriba <[email protected]>
Co-authored-by: Ruben Fiszel <[email protected]>
Co-authored-by: Ruben Fiszel <[email protected]>
Copy link

cloudflare-workers-and-pages bot commented Nov 14, 2024

Deploying windmill with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6c59565
Status:🚫  Build failed.

View logs

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 6c59565 in 13 seconds

More details
  • Looked at 24 lines of code in 1 files
  • Skipped 1 files when reviewing.
  • Skipped posting 3 drafted comments based on config settings.
1. frontend/package.json:106
  • Draft comment:
    Ensure that the updated version of ag-grid-community is compatible with your existing code. Test thoroughly to avoid any breaking changes.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The PR updates dependencies to fix vulnerabilities. It's important to ensure that the updated versions are compatible with the rest of the codebase.
2. frontend/package.json:107
  • Draft comment:
    Ensure that the updated version of ag-grid-enterprise is compatible with your existing code. Test thoroughly to avoid any breaking changes.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The PR updates dependencies to fix vulnerabilities. It's important to ensure that the updated versions are compatible with the rest of the codebase.
3. frontend/package.json:131
  • Draft comment:
    Ensure that the updated version of pdfjs-dist is compatible with your existing code. Test thoroughly to avoid any breaking changes.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The PR updates dependencies to fix vulnerabilities. It's important to ensure that the updated versions are compatible with the rest of the codebase.

Workflow ID: wflow_yT0cLpclIw7EuXJU


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

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