Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Allow @ in X-Request-Id header | Daniel Colson | 2018-01-29 | 1 | -1/+1 |
| | | | | | | | It makes sense to be as strict as possible with headers from the outside world, but allowing @ to support Apache's mod_unique_id (see #31644) seems OK to me | ||||
* | 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 |
| | |||||
* | [ci-skip] Swap method and its alias, format doc | Andrey Molchanov | 2016-10-04 | 1 | -3/+4 |
| | |||||
* | applies new string literal convention in actionpack/lib | Xavier Noria | 2016-08-06 | 1 | -2/+2 |
| | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. | ||||
* | add a branch to eliminate multiple nil checks | Aaron Patterson | 2015-05-23 | 1 | -3/+5 |
| | | | | | if we add an else conditional to the `presence` check, we can eliminate the second `||` branch in the caller | ||||
* | move request id manipulation to the request object | Aaron Patterson | 2015-05-22 | 1 | -6/+5 |
| | | | | this way we can keep the knowledge of `env` hash keys in one place. | ||||
* | Aliased the ActionDispatch::Request#uuid method with ↵ | David Ilizarov | 2015-02-20 | 1 | -1/+1 |
| | | | | ActionDispatch::Request#request_id | ||||
* | Freeze strings to reduce allocations in integration tests | eileencodes | 2015-02-18 | 1 | -4/+8 |
| | | | | | | | | | | | | | Moves `X-Request-ID`, `action_dispatch.request_id` and `HTTP_X_REQUEST_ID` strings to constants and freezes them. We are freezing these strings to reduce the number of allocations in Rails integration tests. The tests are spending a lot of time in GC and this reduces the amount of time spent from 12% to 9% (in combination with Rack PR that also freezes some strings). Number of allocations before this change: 1030722 Number of allocations after this change: 967722 | ||||
* | use 'based on' instead of 'based off' [ci skip] | Akshay Vishnoi | 2014-08-12 | 1 | -1/+1 |
| | |||||
* | Mark unused variables and make some style fixes | Agis Anastasopoulos | 2013-04-08 | 1 | -1/+1 |
| | | | | It'd be a nice convention to mark the unused variables like this, now that Ruby 2 will issue no warnings for such vars being unused. | ||||
* | load active_support/core_ext/object/blank in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | Tap, tap goes the middleware | Jeremy Kemper | 2012-02-29 | 1 | -4/+1 |
| | |||||
* | Using uuid which is supported in 1.9 | Arun Agrawal | 2011-12-21 | 1 | -1/+1 |
| | | | Orignal commit #ada78066fdbccffb1da092a2470211fa252b3c99 | ||||
* | Remove the unneeded `\d` when sanitizing `X-Request-Id`. | Marc Bowes | 2011-10-20 | 1 | -1/+1 |
| | |||||
* | Load object/blank and make use of presence. | José Valim | 2011-10-19 | 1 | -2/+3 |
| | |||||
* | Make tests run on 1.8.x, add integration setup. | José Valim | 2011-10-19 | 1 | -3/+2 |
| | |||||
* | Blah, SecureRandom#uuid is not supported in 1.8.7 -- cant wait for Rails 4.0 ↵ | David Heinemeier Hansson | 2011-10-19 | 1 | -1/+1 |
| | | | | to drop compatibility with 1.8.x | ||||
* | Require missing string access dependency. | José Valim | 2011-10-19 | 1 | -0/+1 |
| | |||||
* | Require securerandom as it is the proper dependency. | José Valim | 2011-10-19 | 1 | -1/+1 |
| | |||||
* | Added X-Request-Id tracking and TaggedLogging to easily log that and other ↵ | David Heinemeier Hansson | 2011-10-19 | 1 | -0/+38 |
production concerns |