| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Enable `Layout/EmptyLinesAroundAccessModifier` cop | Ryuta Kamizono | 2019-06-13 | 1 | -1/+0 |
| | | | | | | | | | | | | We sometimes say "✂️ newline after `private`" in a code review (e.g. https://github.com/rails/rails/pull/18546#discussion_r23188776, https://github.com/rails/rails/pull/34832#discussion_r244847195). Now `Layout/EmptyLinesAroundAccessModifier` cop have new enforced style `EnforcedStyle: only_before` (https://github.com/rubocop-hq/rubocop/pull/7059). That cop and enforced style will reduce the our code review cost. | ||||
| * | Add the `Mime::Type::InvalidMimeType` error in the default rescue_response: | Edouard CHIN | 2019-03-26 | 1 | -2/+6 |
| | | | | | | | | | | | | | | | | - https://github.com/rails/rails/pull/35604 introduced a vulnerability fix to raise an error in case the `HTTP_ACCEPT` headers contains malformated mime type. This will cause applications to throw a 500 if a User Agent sends an invalid header. This PR adds the `InvalidMimeType` in the default `rescue_responses` from the ExceptionWrapper and will return a 406. I looked up the HTTP/1.1 RFC and it doesn't stand what should be returned when the UA sends malformated mime type. Decided to get 406 as it seemed to be the status the better suited for this. | ||||
| * | Use tt in doc for ActionPack [ci skip] | Yoshiyuki Hirano | 2017-08-26 | 1 | -4/+4 |
| | | |||||
| * | Use frozen string literal in actionpack/ | Kir Shatrov | 2017-07-29 | 1 | -0/+2 |
| | | |||||
| * | Revert "Merge pull request #29540 from kirs/rubocop-frozen-string" | Matthew Draper | 2017-07-02 | 1 | -1/+0 |
| | | | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa. | ||||
| * | Enforce frozen string in Rubocop | Kir Shatrov | 2017-07-01 | 1 | -0/+1 |
| | | |||||
| * | Add three new rubocop rules | Rafael Mendonça França | 2016-08-16 | 1 | -2/+2 |
| | | | | | | | | | Style/SpaceBeforeBlockBraces Style/SpaceInsideBlockBraces Style/SpaceInsideHashLiteralBraces Fix all violations in the repository. | ||||
| * | normalizes indentation and whitespace across the project | Xavier Noria | 2016-08-06 | 1 | -19/+19 |
| | | |||||
| * | modernizes hash syntax in actionpack | Xavier Noria | 2016-08-06 | 1 | -1/+1 |
| | | |||||
| * | applies new string literal convention in actionpack/lib | Xavier Noria | 2016-08-06 | 1 | -3/+3 |
| | | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. | ||||
| * | use `Request#path_info` instead of direct ENV access | Aaron Patterson | 2015-08-23 | 1 | -1/+1 |
| | | | | | | we already have a request, so we should use the methods on the request to access the path info information | ||||
| * | Fix ActionDispatch::PublicExceptions returning string rack status | Ryan Tomayko | 2015-03-23 | 1 | -2/+2 |
| | | | | | | | | | The status returned in the rack [status, headers, body] array was a string, which can cause problems with middleware that assumes the status will be a Fixnum. This likely never surfaced because other middleware to_i the status returned from downstream apps before passing it on. | ||||
| * | Uppercase HTML in docs. | Hendy Tanata | 2014-08-08 | 1 | -1/+1 |
| | | | | | [skip ci] | ||||
| * | [ci skip] Document PublicExceptions middleware | schneems | 2014-08-05 | 1 | -0/+10 |
| | | |||||
| * | Remove redundant code. | Guo Xiang Tan | 2014-05-20 | 1 | -3/+2 |
| | | |||||
| * | remove variable and fix warning | Vipul A M | 2013-05-09 | 1 | -1/+0 |
| | | |||||
| * | Fix that JSON and XML exception responses should give the HTTP error message ↵ | Jeremy Kemper | 2013-05-08 | 1 | -2/+2 |
| | | | | | for their status, by default, not the message from the underlying exception | ||||
| * | Use status, content_type, body method signature for render too | Santiago Pastorino | 2012-06-14 | 1 | -4/+3 |
| | | |||||
| * | Refactor public exceptions to reuse render format method | Carlos Antonio da Silva | 2012-06-14 | 1 | -9/+6 |
| | | |||||
| * | content_type is already a Mime::Type object | Santiago Pastorino | 2012-06-12 | 1 | -1/+1 |
| | | |||||
| * | This consider_all_requests_local doesn't make sense | Santiago Pastorino | 2012-06-12 | 1 | -3/+2 |
| | | | | | | This middleware is only for Public Exceptions. This follows bd8c0b8a | ||||
| * | Return proper format on exceptions | Santiago Pastorino | 2012-06-11 | 1 | -14/+36 |
| | | |||||
| * | ShowExceptions should understand X-Cascade responses from exceptions app. | José Valim | 2011-12-16 | 1 | -1/+1 |
| | | |||||
| * | Extract the rendering of public exceptions pages into a Rack app. | José Valim | 2011-12-16 | 1 | -0/+30 |
 |
index : rails.git | |
| Mirror of official rails repo with custom fixes. | Harald Eilertsen |
| aboutsummaryrefslogtreecommitdiffstats |