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

gomod: Update dependencies #893

Merged
merged 3 commits into from
Jan 9, 2025
Merged

Conversation

alexandrevicenzi
Copy link
Contributor

gopkg.in/antchfx/htmlquery.v1 v1.2.2 is affected by GO-2024-3333, bumping to v.1.3.4 fixes the security issue.

@stgraber
Copy link
Member

stgraber commented Jan 8, 2025

Not really an issue though as the CVE is about causing a DoS attack, typically on an online service.
In the case of distrobuilder, it only accesses a pre-defined set of trusted sources and it's not a daemon or other service that can really be DoSed.

@stgraber
Copy link
Member

stgraber commented Jan 8, 2025

Looks like there's a syntax issue with the import changes.

Can you also split things to have the change to the import (gopkg -> github) be its own commit so it's nicely split from the mechanical update of the go.mod.

@alexandrevicenzi
Copy link
Contributor Author

@stgraber Thanks for reviewing. Sorry that I missed the static checks, it should be properly formatted now.

stgraber and others added 3 commits January 9, 2025 11:09
replace gopkg.in/antchfx/htmlquery.v1 with github.com/antchfx/htmlquery

Signed-off-by: Alexandre Vicenzi <[email protected]>
gopkg.in/antchfx/htmlquery.v1 v1.2.2 is affected by GO-2024-3333,
bumping to v.1.3.4 fixes the security issue.

Signed-off-by: Alexandre Vicenzi <[email protected]>
@stgraber stgraber merged commit c438f35 into lxc:main Jan 9, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants