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

Bumping x/crypto to 0.31.0 due to CVE-2024-45337 and running go mod tidy #155

Merged
merged 3 commits into from
Feb 7, 2025

Conversation

nicon89
Copy link
Contributor

@nicon89 nicon89 commented Jan 28, 2025

@nicon89 nicon89 mentioned this pull request Jan 28, 2025
@nicon89
Copy link
Contributor Author

nicon89 commented Jan 28, 2025

@maxisam I guess you need to generate also a new maxisam/mongo-tool image.

@maxisam maxisam merged commit 44fd3d1 into maxisam:main Feb 7, 2025
1 check failed
@maxisam
Copy link
Owner

maxisam commented Feb 8, 2025

Sorry for taking so long. Pls let me know if this work maxisam/mgob:dev.333 , I will create a release once I get feedback

Thanks again!

@nicon89
Copy link
Contributor Author

nicon89 commented Feb 10, 2025

@maxisam doesn't seem to work with gcloud:

time="2025-02-10T17:58:40Z" level=fatal msg="gcloud failed Error loading shared library libpython3.11.so.1.0: No such file or directory (needed by /usr/bin/python3) Error relocating /usr/bin/python3: Py_BytesMain: symbol not found : exit status 127"

@nicon89
Copy link
Contributor Author

nicon89 commented Feb 10, 2025

@maxisam please ignore last comment. I've tested it out on gcloud and it works fine.

@maxisam
Copy link
Owner

maxisam commented Feb 10, 2025

Thanks! I have a e2e test for gcloud as well. So do you think it is good for release?

@nicon89
Copy link
Contributor Author

nicon89 commented Feb 10, 2025

Trivy doesn't report CVE-2024-45337 anymore, so LGTM :)

Repository owner deleted a comment from Sam-Lin-MillersLab Feb 11, 2025
@maxisam
Copy link
Owner

maxisam commented Feb 11, 2025

Thanks for the contribution 😊

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