Skip to content

Fix 500 errors when handling unsafe redirects#6682

Merged
mitchellhenke merged 2 commits intomainfrom
mitchellhenke/fix-unsafe-redirect-500
Aug 3, 2022
Merged

Fix 500 errors when handling unsafe redirects#6682
mitchellhenke merged 2 commits intomainfrom
mitchellhenke/fix-unsafe-redirect-500

Conversation

@mitchellhenke
Copy link
Contributor

@mitchellhenke mitchellhenke commented Aug 2, 2022

Seeing a couple of 500s in staging and int that appear to be partly a result of the Rails 7 upgrade

@mitchellhenke mitchellhenke changed the title Fix 500 error when handling unsafe redirects Fix 500 errors when handling unsafe redirects Aug 2, 2022
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@mitchellhenke mitchellhenke force-pushed the mitchellhenke/fix-unsafe-redirect-500 branch from 80d78a9 to 7efdcb4 Compare August 3, 2022 14:06
Mitchell Henke and others added 2 commits August 3, 2022 09:39
changelog: Bug Fixes, Security, Fix 500 error when handling unsafe redirects
Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
@mitchellhenke mitchellhenke force-pushed the mitchellhenke/fix-unsafe-redirect-500 branch from 7efdcb4 to d8f31ee Compare August 3, 2022 14:40
@mitchellhenke mitchellhenke merged commit 9b192cd into main Aug 3, 2022
@mitchellhenke mitchellhenke deleted the mitchellhenke/fix-unsafe-redirect-500 branch August 3, 2022 15:18
@solipet solipet mentioned this pull request Aug 9, 2022
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.

3 participants