Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Introduce `Headers#add`. Move `Response#add_header` upstream. | Jeremy Daer | 2015-10-03 | 1 | -0/+5 |
| | | | | | | | * Introduce `ActionDispatch::Http::Headers#add` to add a value to a multivalued header. * Move `Response#add_header` upstream: https://github.com/rack/rack/pull/957 * Match upstream `Response#have_header?` -> `#has_header?` name change. | ||||
* | stop inheriting from Rack::Request | Aaron Patterson | 2015-09-04 | 1 | -1/+1 |
| | | | | | | Just include the modules necessary in the Request object to implement the things we need. This should make it easier to build delegate request objects because the API is smaller | ||||
* | use methods on the request object to implement `fetch` | Aaron Patterson | 2015-08-21 | 1 | -2/+8 |
| | | | | Now the Headers internals don't depend on the env hash. | ||||
* | dup the env hash on Header#env | Aaron Patterson | 2015-08-21 | 1 | -1/+1 |
| | | | | | This prevents external mutations from impacting the internals of the request or the Header object. | ||||
* | use accessors on the request object for manipulating env | Aaron Patterson | 2015-08-21 | 1 | -3/+3 |
| | | | | | this reduces the API footprint for the env hash so that we can be more flexible when changing API in the future | ||||
* | use `set_header` rather than []= | Aaron Patterson | 2015-08-21 | 1 | -1/+1 |
| | | | | This allows us to avoid calling `env_name` twice. | ||||
* | dup the request and mutate its headers object. | Aaron Patterson | 2015-08-21 | 1 | -2/+2 |
| | | | | duping the request will dup it's underlying env hash. | ||||
* | pass a request object to the headers object | Aaron Patterson | 2015-08-21 | 1 | -9/+15 |
| | |||||
* | Use a frozen Set instance for CGI_VARIABLES. | Nathaniel Bibler | 2014-06-05 | 1 | -8/+21 |
| | | | Also expand the CGI_VARIABLE name listing to multiple lines for cleaner diffs and legibility. | ||||
* | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2014-05-10 | 1 | -1/+22 |
|\ | |||||
| * | copy edits [ci skip] | Vijay Dev | 2014-05-10 | 1 | -8/+8 |
| | | |||||
| * | [ci skip] doc Http::Headers methods | schneems | 2014-05-09 | 1 | -1/+17 |
| | | |||||
| * | copy edits [ci skip] | Vijay Dev | 2014-05-08 | 1 | -2/+1 |
| | | |||||
| * | [ci skip] document ActionDispatch::HTTP::Headers | schneems | 2014-05-07 | 1 | -0/+6 |
| | | |||||
* | | HTTP::Headers#key? correctly converts | schneems | 2014-05-07 | 1 | -1/+3 |
|/ | | | | | | | | | | | | | | | Previously if you were looking for a given key, the header may incorrectly tell you that it did not exist even though it would return a valid value: ```ruby env = { "CONTENT_TYPE" => "text/plain" } headers = ActionDispatch::Http::Headers.new(env) headers["Content-Type"] # => "text/plain" headers.key?("Content-Type") # => false ``` This PR fixes that behavior by converting the key before checking for presence | ||||
* | `Http::Headers` directly modifies the passed environment. | Yves Senn | 2013-03-15 | 1 | -2/+1 |
| | | | | | | | | | The env hash passed to `Http::Headers#new` must be in env format. Also be aware that the passed hash is modified directly. docs and test-cases for setting headers/env in functional tests. Follow up to #9700. | ||||
* | `Http::Headers` respects dotted env vars, symbols, headers with numbers. | Yves Senn | 2013-03-13 | 1 | -11/+6 |
| | |||||
* | allow headers and env to be passed in `IntegrationTest`. | Yves Senn | 2013-03-13 | 1 | -1/+15 |
| | | | | Closes #6513. | ||||
* | refactor, `Http::Headers` stores headers in env notation | Yves Senn | 2013-03-13 | 1 | -17/+21 |
| | | | | | Also: cleanup, use consistent syntax for `Http::Header` and test. | ||||
* | Http::Headers respects headers that are not prefixed with HTTP_ | Yves Senn | 2013-03-13 | 1 | -1/+13 |
| | |||||
* | move cache inside the instance so we do not need locking | Aaron Patterson | 2012-10-18 | 1 | -9/+12 |
| | |||||
* | add test for fetch with a block | Aaron Patterson | 2012-10-18 | 1 | -2/+2 |
| | |||||
* | prefer composition over inheritence | Aaron Patterson | 2012-10-18 | 1 | -11/+15 |
| | |||||
* | Enable ActionDispatch::Http::Headers to support fetch | Mark Turner | 2012-05-02 | 1 | -7/+8 |
| | |||||
* | Unneeded require memoizable | Akira Matsuda | 2011-11-12 | 1 | -2/+0 |
| | |||||
* | Remove usage of memoizable from ActionPack. | José Valim | 2011-06-16 | 1 | -3/+3 |
| | |||||
* | Silence some trivial warnings: shadowed local vars, indentation mismatches | Jeremy Kemper | 2009-12-28 | 1 | -7/+7 |
| | |||||
* | Move HTTP libs and middleware into ActionDispatch component | Joshua Peek | 2009-01-27 | 1 | -0/+33 |