Skip to content

Conversation

ezekg
Copy link
Contributor

@ezekg ezekg commented Oct 12, 2023

Follow up to #49587. Please let me know if anything is incorrect here. I'm still piecing together the situation discussed in #42922 and #48204, so if @boomer196 and @jorgemanrubia could review as well, that'd be great. I want to make sure the information here is as accurate as possible since this could cause a production outage if not handled correctly.

@rails-bot rails-bot bot added the docs label Oct 12, 2023
@rafaelfranca rafaelfranca added this to the 7.1.2 milestone Oct 12, 2023
@rafaelfranca
Copy link
Member

Thanks! I asked @jorgemanrubia's review

@boomer196
Copy link

It looks accurate to me. Thanks!

Copy link
Contributor

@adrianna-chang-shopify adrianna-chang-shopify left a comment

Choose a reason for hiding this comment

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

Thanks for adding to the docs! ❤️

@ezekg ezekg force-pushed the fix/note-on-rails-7-1-sha-1-bug branch from 4abe47d to 88ed242 Compare October 12, 2023 19:37
Copy link
Contributor

@jorgemanrubia jorgemanrubia left a comment

Choose a reason for hiding this comment

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

This is great. Thanks for adding it.

@rafaelfranca rafaelfranca merged commit 653725e into rails:main Oct 13, 2023
rafaelfranca added a commit that referenced this pull request Oct 13, 2023
Add note on active_record.encryption.support_sha1_for_non_deterministic_encryption
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants