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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bumps rails from 4.2.0 -> 4.2.4 #217

Merged
merged 2 commits into from Mar 7, 2016
Merged

Bumps rails from 4.2.0 -> 4.2.4 #217

merged 2 commits into from Mar 7, 2016

Conversation

@dahlbyk
Copy link
Member

dahlbyk commented Mar 3, 2016

Pulling this out of #211 so we can 馃殺 it separately.

@dahlbyk dahlbyk deployed to huboard-rails-pr-217 Mar 3, 2016 Active
nokogiri (1.6.6.2)
mini_portile (~> 0.6.0)
nokogiri (1.6.7.2)
mini_portile2 (~> 2.0.0.rc2)

This comment has been minimized.

Copy link
@dahlbyk

dahlbyk Mar 3, 2016

Author Member

Any idea why this is 2.0.0.rc2 when version is 2.0.0 above?

@discorick discorick deployed to huboard-rails-pr-217 Mar 7, 2016 Active
@discorick

This comment has been minimized.

Copy link
Member

discorick commented Mar 7, 2016

Ok @dahlbyk I am pretty confident wit this one now - I ran into a problem with arguments not serializing/deserializing between ActiveJobs and it took me way to long to figure out what was wrong.

Essentially plain Hash's are now being transported as such, before this upgrade they were being automatically coalesced into HashWithIndifferentAccess

See c02ed1a

@seanlinsley

This comment has been minimized.

Copy link

seanlinsley commented Mar 7, 2016

Y'all should upgrade to 4.2.5.2 soon. It includes a number of security fixes.

@dahlbyk

This comment has been minimized.

Copy link
Member Author

dahlbyk commented Mar 7, 2016

See c02ed1a

馃憤

Y'all should upgrade to 4.2.5.2 soon. It includes a number of security fixes.

@discorick thoughts on doing this now while we're already bumping versions?

@discorick

This comment has been minimized.

Copy link
Member

discorick commented Mar 7, 2016

@seanlinsley @dahlbyk I would like to get this guy merged in first to unblock some PR's in flight. But point taken, +1 for security fixes, we should look into 4.2.5.2 as soon as we can

discorick added a commit that referenced this pull request Mar 7, 2016
Bumps rails from 4.2.0 -> 4.2.4
@discorick discorick merged commit 3934e45 into master Mar 7, 2016
@discorick discorick deleted the rails-4.2.4 branch Mar 7, 2016
@dahlbyk dahlbyk mentioned this pull request Mar 7, 2016
@dahlbyk

This comment has been minimized.

Copy link
Member Author

dahlbyk commented Mar 7, 2016

But point taken, +1 for security fixes, we should look into 4.2.5.2 as soon as we can

Issue created. Thanks for the tip @seanlinsley!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can鈥檛 perform that action at this time.