Skip to content

[authutil] standardize argon2 dependency#244

Merged
cviecco merged 2 commits intoCloud-Foundations:masterfrom
sethyes:authutil/standardize_argon2_dependency
Oct 29, 2024
Merged

[authutil] standardize argon2 dependency#244
cviecco merged 2 commits intoCloud-Foundations:masterfrom
sethyes:authutil/standardize_argon2_dependency

Conversation

@sethyes
Copy link
Copy Markdown
Contributor

@sethyes sethyes commented Sep 5, 2024

Utilize the argon2 package in golang.org/x. This function doesn't return an error, so minor cleanup is conducted surrounding error handling as well.

github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
github.com/dchest/blake2b v1.0.0 h1:KK9LimVmE0MjRl9095XJmKqZ+iLxWATvlcpVFRtaw6s=
github.com/dchest/blake2b v1.0.0/go.mod h1:U034kXgbJpCle2wSk5ybGIVhOSHCVLMDqOzcPEA0F7s=
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

These deps were removed as a result of running go mod tidy.

@cviecco cviecco self-requested a review October 29, 2024 18:48
@cviecco
Copy link
Copy Markdown
Contributor

cviecco commented Oct 29, 2024

LGTM, thank you for your contribution

Copy link
Copy Markdown
Contributor

@cviecco cviecco left a comment

Choose a reason for hiding this comment

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

LGTM, did an extra fix matching go mod updates.

@cviecco cviecco merged commit 2f98c62 into Cloud-Foundations:master Oct 29, 2024
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.

2 participants