Skip to content

Updated rails#145

Merged
steveklabnik merged 1 commit intohacketyhack:masterfrom
jrgifford:update-rails-for-another-security-problem
Feb 11, 2013
Merged

Updated rails#145
steveklabnik merged 1 commit intohacketyhack:masterfrom
jrgifford:update-rails-for-another-security-problem

Conversation

@jrgifford
Copy link
Copy Markdown
Member

more security holes plugged.

steveklabnik added a commit that referenced this pull request Feb 11, 2013
@steveklabnik steveklabnik merged commit cd34a5a into hacketyhack:master Feb 11, 2013
@steveklabnik
Copy link
Copy Markdown
Member

Thanks! Deploying now.

@jrgifford
Copy link
Copy Markdown
Member Author

Awesome. I had two failing tests locally, otherwise I would have just pushed straight.

@jrgifford jrgifford deleted the update-rails-for-another-security-problem branch February 11, 2013 21:14
@jrgifford jrgifford restored the update-rails-for-another-security-problem branch February 11, 2013 21:14
@jrgifford jrgifford deleted the update-rails-for-another-security-problem branch February 11, 2013 21:14
@steveklabnik
Copy link
Copy Markdown
Member

Oh? I didn't even look tbh. We should check that out then.

@jrgifford
Copy link
Copy Markdown
Member Author

they all pass on travis though, so i wasn't concerned.

@jrgifford
Copy link
Copy Markdown
Member Author

(i'm learning how to develop on fedora instead of ubuntu, so i'm probably missing packages that are required)

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.

2 participants