2 ms·
Dan: Thanks again for your work (I also work on BadgeApp)! One other important change due to the Rails 5 upgrade was that because jquery-turbolinks doesn't sup
by dwheeler 10y ago
Dan: Thanks again for your work (I also work on BadgeApp)!
One other important change due to the Rails 5 upgrade was that because jquery-turbolinks doesn't support Rails 5 [0], we eventually had to drop both jquery-turbolinks and turbolinks. If you're using jquery-turbolinks, you may also need to drop both jquery-turbolinks and turbolinks. Turbolinks doesn't work correctly with many JavaScript libraries (causing no end of subtle bugs), and in our case the performance boost turbolinks is supposed to provide, so turbolinks just wasn't worth the trouble anyway. Good riddance.
Rails 5 has some nice additions. I particularly appreciate that Rails 5 has two new security hardening mechanisms to reduce the risk of CSRF security vulnerabilities: per-form CSRF tokens and origin-checking CSRF mitigation. We use a number of mechanisms to make our application secure [1].
The application doesn't have many LOC, but we do work to keep it that way. We try to reuse code where it makes sense, and always try to keep it simple. Still, it doesn't take many lines to create a useful app, and that's a good thing.
[0] https://github.com/kossnocorp/jquery.turbolinks/issues/56 https://github.com/kossnocorp/jquery.turbolinks/issues/56
[1] https://github.com/linuxfoundation/cii-best-practices-badge/blob/master/doc/security.md https://github.com/linuxfoundation/cii-best-practices-badge/...