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

[UPD] Severity Critical 9.8 / 10 - v3.3.1 Fix CRITICAL security issue GHSA-h755-8qp9-cq85 protobufjs #253

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

odselsevier
Copy link
Contributor

@odselsevier odselsevier commented Sep 21, 2023

3.3.1 Fix CRITICAL security issue GHSA-h755-8qp9-cq85
related to d2367bd

@odselsevier odselsevier changed the title 3.3.1 Fix CRITICAL security issue [GHSA-h755-8qp9-cq85](https://github.com/advisories/GHSA-h755-8qp9-cq85) 3.3.1 Fix CRITICAL security issue GHSA-h755-8qp9-cq85 Sep 21, 2023
@odselsevier odselsevier changed the title 3.3.1 Fix CRITICAL security issue GHSA-h755-8qp9-cq85 Severity Critical 9.8 / 10 - v3.3.1 Fix CRITICAL security issue GHSA-h755-8qp9-cq85 protobufjs Oct 5, 2023
@odselsevier
Copy link
Contributor Author

@Nevon feel free to merge and publish this security issue fix release 3.3.1

@odselsevier
Copy link
Contributor Author

@Nevon could you please review pr?

@odselsevier odselsevier changed the title Severity Critical 9.8 / 10 - v3.3.1 Fix CRITICAL security issue GHSA-h755-8qp9-cq85 protobufjs [UPD] Severity Critical 9.8 / 10 - v3.3.1 Fix CRITICAL security issue GHSA-h755-8qp9-cq85 protobufjs Mar 7, 2024
@odselsevier
Copy link
Contributor Author

@leow93 @andreacioni @Nevon @Dieken could you please review merge pr? it fixing critical issue with protobuf library

@Dieken
Copy link
Contributor

Dieken commented Mar 7, 2024

LGTM, but I don't have permission to merge.

@andreacioni
Copy link

LGTM, but no write permission for me neither. I'd really appreciate a release for this too.

Copy link

@leow93 leow93 left a comment

Choose a reason for hiding this comment

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

Likewise, no merge permissions but it looks good.

@odselsevier
Copy link
Contributor Author

maybe @Nevon can help us with that :)

@odselsevier
Copy link
Contributor Author

@Nevon could you merge?

@odselsevier
Copy link
Contributor Author

@tulios could you merge?

@odselsevier odselsevier marked this pull request as draft July 24, 2024 21:17
@odselsevier odselsevier marked this pull request as ready for review July 24, 2024 21:17
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.

4 participants