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

Add CVE-2021-32821 for GHSL-2020-345 #8443

Merged
merged 1 commit into from
Jan 3, 2023

Conversation

shelbyc
Copy link
Contributor

@shelbyc shelbyc commented Jan 3, 2023

Add CVE-2021-32821 for GHSL-2020-345

@shelbyc shelbyc requested a review from cve-team as a code owner January 3, 2023 16:48
@cve-team cve-team self-assigned this Jan 3, 2023
@cve-team cve-team merged commit 66d294f into CVEProject:master Jan 3, 2023
@Marcono1234
Copy link

It looks like this by accident added a .DS_Store file to the repository. Should this be reverted?

@shelbyc
Copy link
Contributor Author

shelbyc commented Sep 11, 2023

@Marcono1234 Thanks for letting me know about the .DS_Store file! The .DS_Store file should be removed.

@Marcono1234
Copy link

Are you planning on doing this, and do you have a direct connection to some of the project maintainers? Because it looks recent PRs weren't answered, and maybe they don't accept regular PRs anymore.

But on the other hand this repository might become obsolete in a few months anyway (see #8855), so in that case it might not matter if the .DS_Store file remains.

@shelbyc shelbyc mentioned this pull request Sep 11, 2023
@shelbyc
Copy link
Contributor Author

shelbyc commented Sep 11, 2023

I'm not sure if the PR will go through, but it changes an extraneous file rather than a CVE record, so we'll see.

@davidaucoin7377
Copy link

Add CVE-2021-32821 for GHSL-2020-345

Copy link

@davidaucoin7377 davidaucoin7377 left a comment

Choose a reason for hiding this comment

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

This needs review before finalizing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants