Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ColinDKelley
Copy link
Collaborator

@ColinDKelley ColinDKelley commented Jul 20, 2021

Addresses issue #452 by using SHA256 instead of MD5.

Copy link
Member

@ioquatix ioquatix left a comment

Choose a reason for hiding this comment

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

Oh no.. MD5.

@ColinDKelley
Copy link
Collaborator Author

Thanks @ioquatix.

What do you think about version bumping for this? It's not a major version bump, since no interfaces changed. But the dependencies did change slightly, so probably a minor version bump?

@ioquatix
Copy link
Member

ioquatix commented Jul 20, 2021

I think it's fine. No one should be depending on the checksum implementation. I'd even just do a patch release.

@ColinDKelley
Copy link
Collaborator Author

No one should be depending on the checksum implementation.

I agree they shouldn't. But if you told me that Truffle- or J- or MRI Ruby version X has a problem where the SHA256 interface is broken in some way (perhaps because of libssl?) I'd believe you in an instant. I've started leaning towards minor version bumps for dependency changes for this reason.

But here's an even better reason: this release actually adds a feature: FIPS compatibility.

@ColinDKelley ColinDKelley merged commit 2693d44 into guard:master Jul 20, 2021
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.

Listen's use of Digest::MD5 breaks support on FIPS enabled machines

2 participants