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

(docs) update filebucket doc to reflect cryptographic checksum instead of md5 #9117

Merged

Conversation

golflimaechoecho
Copy link
Contributor

Default digest_algorithm changed in Puppet 7.0.0; update filebucket documentation to reflect no longer md5 by default.

The algorithm used is specified by the digest_algorithm setting; md5 was the default for Puppet6 and earlier; from Puppet 7.0.0 onwards the default was changed to sha256 - ref: https://www.puppet.com/docs/puppet/7/release_notes_puppet.html#enhancements_puppet_7-0-0-pup-10583

@golflimaechoecho golflimaechoecho requested a review from a team as a code owner October 5, 2023 03:33
@CLAassistant
Copy link

CLAassistant commented Oct 5, 2023

CLA assistant check
All committers have signed the CLA.

@golflimaechoecho golflimaechoecho force-pushed the update_filebucket_doc_comment branch from 186b38f to 6f5e338 Compare October 5, 2023 03:36
Copy link
Contributor

@joshcooper joshcooper left a comment

Choose a reason for hiding this comment

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

@joshcooper joshcooper merged commit 42543dd into puppetlabs:main Oct 5, 2023
9 checks passed
@golflimaechoecho golflimaechoecho deleted the update_filebucket_doc_comment branch October 6, 2023 01:00
@joshcooper joshcooper added the bug Something isn't working label Nov 15, 2023
@joshcooper joshcooper added the backport 7.x Generate a backport PR to 7.x label Dec 2, 2023
Copy link

github-actions bot commented Dec 2, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 7.x Generate a backport PR to 7.x bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants