Skip to content

Conversation

@ljdelavega
Copy link
Contributor

To address the Critical-level CVE:

GHSA-vqfr-h8mv-ghfj

Summary

Checklist

  • I understand that this PR may be closed in case there was no previous discussion. (This doesn't apply to typos!)

Copy link

@lukehsiao lukehsiao left a comment

Choose a reason for hiding this comment

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

My approval means nothing, but thanks for making this PR!

Copy link

@techgaun techgaun left a comment

Choose a reason for hiding this comment

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

came exactly for this, could we get this out asap? would appreciate it :)

Copy link

@kingbuzzman kingbuzzman left a comment

Choose a reason for hiding this comment

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

LGTM. Kinda blocked by this..

@joanise
Copy link

joanise commented Apr 24, 2025

Waiting on this to resolve the issue for our project too. I recommend accepting @lkasser1 's suggestion before merging, but we don't depend on that.

Co-authored-by: Lucas Kasser <lkasser8@gmail.com>
@lovelydinosaur
Copy link
Contributor

Thanks.

Aside; request contributors to not unnecessarily pile on in issues please. The initial prompt is sufficient.

@lovelydinosaur lovelydinosaur merged commit d1e17c5 into encode:master Apr 24, 2025
5 checks passed
@lovelydinosaur lovelydinosaur mentioned this pull request Apr 24, 2025
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.

7 participants