aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch/middleware/request_id.rb
Commit message (Collapse)AuthorAgeFilesLines
* Allow @ in X-Request-Id headerDaniel Colson2018-01-291-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 Shatrov2017-07-291-0/+2
|
* Revert "Merge pull request #29540 from kirs/rubocop-frozen-string"Matthew Draper2017-07-021-1/+0
| | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa.
* Enforce frozen string in RubocopKir Shatrov2017-07-011-0/+1
|
* [ci-skip] Swap method and its alias, format docAndrey Molchanov2016-10-041-3/+4
|
* applies new string literal convention in actionpack/libXavier Noria2016-08-061-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 checksAaron Patterson2015-05-231-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 objectAaron Patterson2015-05-221-6/+5
| | | | this way we can keep the knowledge of `env` hash keys in one place.
* Aliased the ActionDispatch::Request#uuid method with ↵David Ilizarov2015-02-201-1/+1
| | | | ActionDispatch::Request#request_id
* Freeze strings to reduce allocations in integration testseileencodes2015-02-181-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 Vishnoi2014-08-121-1/+1
|
* Mark unused variables and make some style fixesAgis Anastasopoulos2013-04-081-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/railsXavier Noria2012-08-021-1/+0
|
* Tap, tap goes the middlewareJeremy Kemper2012-02-291-4/+1
|
* Using uuid which is supported in 1.9Arun Agrawal2011-12-211-1/+1
| | | Orignal commit #ada78066fdbccffb1da092a2470211fa252b3c99
* Remove the unneeded `\d` when sanitizing `X-Request-Id`.Marc Bowes2011-10-201-1/+1
|
* Load object/blank and make use of presence.José Valim2011-10-191-2/+3
|
* Make tests run on 1.8.x, add integration setup.José Valim2011-10-191-3/+2
|
* Blah, SecureRandom#uuid is not supported in 1.8.7 -- cant wait for Rails 4.0 ↵David Heinemeier Hansson2011-10-191-1/+1
| | | | to drop compatibility with 1.8.x
* Require missing string access dependency.José Valim2011-10-191-0/+1
|
* Require securerandom as it is the proper dependency.José Valim2011-10-191-1/+1
|
* Added X-Request-Id tracking and TaggedLogging to easily log that and other ↵David Heinemeier Hansson2011-10-191-0/+38
production concerns