Update Rails or not – security issues either way

http://www.browserbite.com/rails-update-or-not-security-issues-either-way/

by kasparloog • 13 years ago

42 47 13 years ago

bradleyland3• 13 years ago

Another option is to fork Rails and use the patch files (included with the CVE) that target only the vulnerabilities addressed in the CVE.

The problem with upgrading to mitigate security issues is that the Rails team does not release security patches. They bump the minor-minor and do a release. That release almost always includes commits that are unrelated to the security issue. This is especially true when a lot of time passes between CVEs.

Maintaining your own branch really isn't that difficult, because you can simply merge in from upstream. In most cases, you're really only interested in patching from Rails team issued security fixes, so you won't have conflicts. If you do have conflicts, you can safely overwrite anything in your fork, because you're not developing Rails, you're simply maintaining tighter control over the release that you use.

locofacetwice• 13 years ago

Or don't use Rails.

static_typed4• 13 years ago

^^ This is good advice. Maybe they should question their choice of platform - there are other options, some of which seem to have had a bit more forethought in their architecture and engineering.

Remember: Ruby/Rails to pose, Python for pros.

tptacek8• 13 years ago

Or Linux, or MySQL.

myke_cameron• 13 years ago

No matter your stack, you are occasionally going to have to deal with security vulnerabilities. Thankfully, rails at least is quick to issue releases which address security vulnerabilities. Back when I was working on java applets, I remember security issues in Java plugins which would persist for weeks, months, or even years.

kasparloog• 13 years ago

Well - that's enough work to maintain your own app. Maintainging a fork of Rails and drilling through all of the vulnerabilities - that's an extra overhead. It's simpler to regression test your own app, I suppose.

bradleyland3• 13 years ago

I disagree that it's simpler to regression test your own app. The recent minor-minor version upgrade passed Rails core tests, as well as test suites at high profile Rails users, yet it contained a regression that caused the disclosure of some sensitive issues at those same high profile shops.

It's easier, from my viewpoint, to stick it out at a release that you know works for you, applying patches from the files contained in the CVEs. There's still a chance that the security patch will cause a regression, but at least you're not pulling in all the interim commits.

mratzloff2• 13 years ago

So what was the issue from Rails, specifically?

knowtheory4• 13 years ago

Specifically that the security releases included changes to the way that ActiveRecord works (which were unrelated to the security issues).

As a consequence, their search queries were scoped differently than what they had intended. So their choice was roll back the security release, or modify their app to accommodate ActiveRecord's altered behavior.

mratzloff2• 13 years ago

I see. Thanks for the clarification.

jrochkind12• 13 years ago

Actually, ironically, the _particular_ bug that changed how ActiveRecord works... which caused security problems... was actually an unintentional regression in a _security patch_.

There were ALSO unrelated changes (quite many) in the patch release that included the latest security fixes. Which is a mess.

But, the _particular_ problem here, the OP suggests it was the same problem as github reported [here](https://github.com/blog/1440-today-s-email-incident), where they suggest they isolated the introduction of the regression to [this commit](https://github.com/rails/rails/commit/f980289fd2c1b9073a94b5...), which in fact has a commit message as the fix to CVE-2013-1854.

So, yes, a security fix unintentionally introduced a regression with _other_ security implications. Yeah, this is kind of ironic, and yeah, it means it's not so simple to say what could have been done to avoid it. (In this case, I'm surprised there wasn't an automated test already that caught the particular regression. It seems like something that should have been tested. But I haven't looked at the test source to see if it was an odd edge case or what have you.)

(But it's STILL bad practice to release security patches only in releases bundled with a bunch of other changes).

tomkin2• 13 years ago

Do you know how many Rails developers I've heard pissing on Flash or Java and its security vulnerabilities? Many - until recently. Now suddeningly these faults are an accepted aspect of Rails development. Really. I guess this more of a rant about how unfairly the Rails community (ie. dhh) has been on others, but now expects critics to look away while it happens in the Rails community on a weekly basis.

static_typed4• 13 years ago

Rails became worse than the frameworks and ecosystems it laughed at in it's younger days.

Remember - Rails is Omakase - meaning literally 'leave it to someeone else' - food for thought indeed.

knowtheory4• 13 years ago

Your point (trolling really) doesn't make any sense (and really the Omakase thing never made sense to begin with).

The core value proposition for Rails has always been it incorporates enough of all of the things you need to get a web app up and running quickly, easily, and with sufficient power that your app can continue to grow into the future.

That's what DHH's original blog post screen cast was about certainly. wycats and carllerche's contributions to Rails after the Rails/Merb merge have focused both on better code discipline and ease/simplicity of use for everyone, from beginners to advanced devs.

On top of that the Rails security team has been totally on top of these disclosures and releasing patches that address them. Prominent members of the Rails community have been extraordinarily vocal in advocating that EVERYONE needs to upgrade their apps.

So, please, tell me again how Rails leaves things to others.

P.S. if you really want á la carte, use Sinatra, or Padrino.

P.P.S. Ah, if you look at the actual meaning of omakase (http://en.wikipedia.org/wiki/Omakase ), it basically means, devs entrust the defaults to the Rails team, which is basically how things actually work w/ Rails.

knowtheory4• 13 years ago

I'm guessing this schadenfreude must be delicious, since everyone keeps coming back for additional helpings.

Look, gloating over other people's security issues is shitty behavior, and frankly responsible developers don't do it. Am I calling DHH an irresponsible developer? Yeah. But the Rails core team has thankfully incorporated a number of smart and responsible developers over the past 4 years, and you see a whole lot less shit like that.

Even better, security issues that have languished silently since 2006 are being identified and quickly addressed.

So, no, nobody is asking anybody to look the other way. There are developers who have been in the Ruby and Rails community for a long time, who make no aspirations to being "rockstars" and are out spending their nights and weekends patching rails, and are not slagging other people off for using Java.

Think of them when you feel the inclination to rant.

(also, incidentally, most of the ranting about Flash and Java development have not focused on security issues, they've mainly focused on how awful it is to use those platforms to do development. Which is why things like JRuby are so wonderful. You get to develop in Ruby, and access the JVM's system of libraries.)

tomkin2• 13 years ago

I agree with most of what you said, but you accuse me of not thinking of the RoR developer community, which is false. My issue lies with extremely critical community leaders suddenly disappearing into the shadows. If you beat others up, expect the same, is all. I don't relish the idea of picking apart the community's hard work, but I do expect some fairness in the dialog.

tptacek8• 13 years ago

The security problems of Flash and Java are not comparable to those of Rails. They're different in magnitude, different in number, and different in circumstance and origin.

I strongly agree with 'knowtheory that gloating about security vulnerabilities is a bad habit. But this Rails/Java comparison is even worse. Nobody personalizes Java insecurity. The Java applet plugin is a mess, responsible for a huge number of compromised desktops, but nobody I know would assume that a developer who worked in Java or on the JVM would be security-illiterate. That's not true of the Rails drama, which is really an opportunity for people to piss on DHH and his personality cult, as you can see in this subthread with 'static_typed's comment.

sergiotapia2• 13 years ago

I've shared my sentiment here before on why I'm not going to use Rails anymore. Imagine having your code working fine, and you update a MINOR version 3.2.x - and your ORM starts returning results that it didn't used to.

Heh.

Not worth the heartburn. I've switched to a saner, more tought out framework. Rails is gorgeous, but not safe to use for any serious systems where you're in a small team.

superuser2• 13 years ago

What did you switch to?

innguest• 13 years ago

Probably ASP.Net MVC -> https://news.ycombinator.com/item?id=5410169

sergiotapia2• 13 years ago

ASP.Net MVC4.

philwelch2• 13 years ago

If you're willing to trade off stability for features, the Rails 2.3 line still receives security patches to this day. You can upgrade to Rails 3 when Rails 4 comes out.

If you're willing to make a slightly different trade-off, just apply the security patches as they come out and don't upgrade minor versions without integration testing.

zwily2• 13 years ago

I have heard, but can't find an official reference to it now, that Rails 2.3 will stop receiving security updates once Rails 4 is released. (I suspect that patches will still become available for serious issues even once official support is dropped.)

static_typed4• 13 years ago

Give a developer a Ruby-based web framework, and they can run for a day without patching, but give them anything else - even Prolog on Punchcards - and they run for so much longer!

Stop today, and help recovering Ruby developers onto a better path!

sbank2• 13 years ago

Enjoying your crusade?

PetrolMan• 13 years ago

I'm not really sure I understand this sentiment. I understand that the Rails community has had a reputation in the past of being arrogant but why take joy in the growing pains of a development community? Why foster an "us vs them" sentiment in the general development community? My understanding is that we are all doing the same things, we just have chosen different ways to get there.

davesims• 13 years ago

Your attempt to drive developers to Ruby and Rails by playing the role of a typical ignorant anti-rails fundamentalist crusader will fail.

websitescenes• 13 years ago

There is no such thing as an insurmountable security configuration. No matter what you use or how you use it, there will always be new methods to compromise your data. Hackers are creative and enjoy a challenge and will continually attack systems until the end. Whether it be Rails or another framework, you will have to upgrade and address security issues forever. Developers need be realistic and understand that any system can be compromised if one tries hard enough. I don't necessarily think that you should go from rails 2 to 3 or 3 to 4 just to address security features. Better off staying in the branch you started the project in and applying the applicable patches for said security concerns. With that said, RAILS RULES!!

jrochkind12• 13 years ago

Rails _needs_ to produce security-patch-only patch releases.

This won't be foolproof; there is no such thing as foolproof in software development. There can still be unintentional regressions in security-patch-only releases (and even new security vulnerabilities), as well as new security vulnerabilities in other releases.

Yes, nothing is foolproof. But you can work at improving your quality. Yes, there are other things Rails could do to improve quality, including trying to actually do some variation of semver.

But the most obvious thing with the highest benefit/cost that Rails team could do is always release security patches in security patch only releases, so developers can apply security patch only releases and minimize their exposure to new regressions when doing so. Git should not make this hard to do, just cherry pick the security-patch commits into a branch created off the last release, and release it as a patch release seperately from any other changes.

Am I missing something?

jeremymcanally• 13 years ago

Well, the release policy published by the Rails team states releases like this are supposed to only have security patches, but for some reason it wasn't followed here. I'm not sure why other than the frantic "zomg there's a security hole must fix it!" fray that may have clouded their judgement.

tptacek8• 13 years ago

No question: update Rails.

The security issues attributable to Rails upgrades are small in number and relatively minor.

The security issues attributable to not upgrading Rails, and in particular in not upgrading during upgrade cycles where the Rails community was reassuring itself that everything was fine and that normal apps wouldn't be affected by security flaws, are much larger in number and extremely significant.

If there was one lesson I'd hope people would take away from Rails Winter of Security Madness, it is be ready to patch at all times. That's a good lesson for every platform, not just Rails, but Rails teams now have specific reasons to be on top of this.

When Rails announces security flaws, patch ASAP. If you're a professional Rails team, dry-run this well in advance; know you can patch at a moments notice, don't just hope.

dasil0032• 13 years ago

How do you dry-run a Rails patch? I mean I know how to do `bundle update rails`. The problems with updates tend to be patch-specific.

tptacek8• 13 years ago

If the only mechanism you have to update Rails is to do a "bundle update", start by fixing that.

Update Rails or not – security issues either way

Related Stories