Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Eliminate Rack::File headers deprecation warning | Sam Ruby | 2013-01-08 | 1 | -1/+1 |
| | | | | | | | | | | | | See http://intertwingly.net/projects/AWDwR4/checkdepot/section-6.1.html rake test produces: "Rack::File headers parameter replaces cache_control after Rack 1.5." Despite what the message says, it appears that the hearders parameter change will be effective as of Rack 1.5: https://github.com/rack/rack/blob/rack-1.4/lib/rack/file.rb#L24 https://github.com/rack/rack/blob/master/lib/rack/file.rb#L24 | ||||
* | Avoid Rack security warning no secret provided | Santiago Pastorino | 2013-01-08 | 1 | -0/+2 |
| | | | | This avoids "SECURITY WARNING: No secret option provided to Rack::Session::Cookie." | ||||
* | refactor ShowExceptions' #call to use def-rescue instead of begin-rescue | Gosha Arinich | 2013-01-07 | 1 | -7/+4 |
| | |||||
* | remove begin-rescue in favor of def-rescue | Gosha Arinich | 2013-01-07 | 1 | -10/+9 |
| | |||||
* | Fix operators precedence issue | Rafael Mendonça França | 2013-01-06 | 1 | -1/+1 |
| | |||||
* | Merge pull request #8787 from tank-bohr/master | Rafael Mendonça França | 2013-01-06 | 1 | -2/+2 |
|\ | | | | | masgn and response variable | ||||
| * | return multiple assingment and response variable | tank-bohr | 2013-01-07 | 1 | -2/+2 |
| | | |||||
* | | Merge pull request #8785 from goshakkk/refactor-debug-exceptions | Rafael Mendonça França | 2013-01-06 | 1 | -2/+1 |
|\ \ | |/ |/| | Refactor DebugExceptions | ||||
| * | refactor DebugExceptions by combining two conditionals into one | Gosha Arinich | 2013-01-07 | 1 | -2/+1 |
| | | |||||
* | | Reduce number of Strings a bit | Akira Matsuda | 2013-01-07 | 1 | -3/+3 |
|/ | |||||
* | fix for rbx | tank-bohr | 2013-01-07 | 1 | -2/+2 |
| | | | | | | | | | Rubinius returns a boolean after such assingment response = (_, headers, body = @app.call(env)) see https://github.com/rubinius/rubinius/issues/2117 get rid of a local variable | ||||
* | Rename route_wrapper partial layout to table | Carlos Antonio da Silva | 2013-01-06 | 1 | -0/+0 |
| | | | | | | It is used by the table formatter only, and it's already inside a routes directory that namespaces it properly, so calling it just "table" seems simpler. | ||||
* | Move table routes formatter class to the inspector and rename it | Carlos Antonio da Silva | 2013-01-06 | 2 | -19/+1 |
| | | | | | | | It feels more consistent to have this class called "HtmlTableFormatter", and to have it here with the routes inspector and console formatter, since it's used for both routing error exceptions and the rails info page. | ||||
* | Remove suggestion that Procs can be used as session secrets. | James Coglan | 2013-01-05 | 1 | -7/+4 |
| | |||||
* | Close container div tag in routing error page | Carlos Antonio da Silva | 2013-01-05 | 1 | -9/+11 |
| | |||||
* | Move style to head to make routes page valid html5 | Carlos Antonio da Silva | 2013-01-05 | 2 | -4/+6 |
| | |||||
* | display mountable engine routes on RoutingError. | Yves Senn | 2013-01-05 | 2 | -8/+23 |
| | |||||
* | Allow use of durations for ActionDispatch::SSL configuration | Andrew White | 2013-01-04 | 1 | -1/+1 |
| | |||||
* | Remove unnecessary `ERB::Util::h` | Ryunosuke SATO | 2013-01-05 | 7 | -17/+17 |
| | | | | It is automatically applied when strings is unsafe for html. | ||||
* | Suppress warning about IO#lines in Ruby 2.0 | Ryunosuke SATO | 2013-01-04 | 1 | -1/+1 |
| | | | | actionpack/lib/action_dispatch/middleware/exception_wrapper.rb:99: IO#lines is deprecated; use #each_line instead | ||||
* | move error page js into script tag | Gosha Arinich | 2013-01-04 | 3 | -5/+27 |
| | |||||
* | Merge pull request #8713 from goshakkk/better-error-page | Guillermo Iguaran | 2013-01-03 | 2 | -12/+10 |
|\ | | | | | Fix env toggling, improve error page styling | ||||
| * | fix env toggling, improve error page styling | Gosha Arinich | 2013-01-03 | 2 | -12/+10 |
| | | |||||
* | | Explain the possible precautions | Andre Arko | 2013-01-02 | 1 | -3/+4 |
| | | |||||
* | | Restore original remote_ip algorithm. | Andre Arko | 2013-01-02 | 1 | -53/+112 |
|/ | | | | | | | | | | | Proxy servers add X-Forwarded-For headers, resulting in a list of IPs. We remove trusted IP values, and then take the last given value, assuming that it is the most likely to be the correct, unfaked value. See [1] for a very thorough discussion of why that is the best option we have at the moment. [1]: http://blog.gingerlime.com/2012/rails-ip-spoofing-vulnerabilities-and-protection/ Fixes #7979 | ||||
* | Fixing closing </p> | Guillermo Iguaran | 2013-01-02 | 1 | -1/+1 |
| | |||||
* | Fix a number of validation/style errors: | Sam Ruby | 2013-01-02 | 2 | -8/+7 |
| | | | | | | | | | | * <pre> is not allowed to be nested inside of <p> elements in HTML * Indentation of </p> doesn't match corresponding <p> * <p> element not explicitly closed * One more </div> than <div> In each case, the template was fixed to match how a HTML5 parser would "see" the resulting page. | ||||
* | Merge pull request #8688 from goshakkk/error-page-toggle | Guillermo Iguaran | 2013-01-02 | 1 | -3/+3 |
|\ | | | | | Allow toggling dumps on error page | ||||
| * | allow toggling dumps instead of just showing | Gosha Arinich | 2013-01-02 | 1 | -3/+3 |
| | | |||||
* | | Cleanup some unnecessary CSS on the new error page and reformat some lines. | Lucas Mazza | 2013-01-02 | 1 | -12/+13 |
| | | |||||
* | | add source line padding | Gosha Arinich | 2013-01-02 | 1 | -0/+4 |
|/ | |||||
* | remove meaningless AS::FrozenObjectError | Akira Matsuda | 2013-01-02 | 1 | -1/+0 |
| | |||||
* | Fix indent in UnknownAction template | Guillermo Iguaran | 2013-01-01 | 1 | -1/+1 |
| | |||||
* | Summary and Details HTML elements aren't supported in all modern browsers | Guillermo Iguaran | 2012-12-31 | 2 | -12/+12 |
| | |||||
* | Add style to AV::Template::Error exception page | Guillermo Iguaran | 2012-12-31 | 2 | -14/+42 |
| | |||||
* | Improve line-height to have better line spacing in exception message | Guillermo Iguaran | 2012-12-31 | 1 | -0/+1 |
| | |||||
* | Add new style to Routing Error page | Guillermo Iguaran | 2012-12-31 | 1 | -19/+22 |
| | |||||
* | Styling for exception page | Guillermo Iguaran | 2012-12-31 | 6 | -29/+150 |
| | |||||
* | Add source extract to detailed exception page | Guillermo Iguaran | 2012-12-31 | 4 | -2/+34 |
| | |||||
* | Define [], []=, permanent, signed and encrypted as the only allowed methods ↵ | Santiago Pastorino | 2012-12-30 | 1 | -9/+57 |
| | | | | for the non Raw Cookie classes | ||||
* | We need to check explictly that env['action_dispatch.show_exceptions'] | Rafael Mendonça França | 2012-12-26 | 1 | -1/+1 |
| | | | | | | is false. If it is nil we can't raise the exception | ||||
* | more descriptive return parameters | Hrvoje Šimić | 2012-12-26 | 1 | -4/+3 |
| | |||||
* | Minor js review [ci skip] | Carlos Antonio da Silva | 2012-12-18 | 1 | -7/+7 |
| | | | | Remove/add ; where necessary, fix indentation. | ||||
* | Format routes as html on debug page | schneems | 2012-12-17 | 4 | -2/+76 |
| | | | | | | | | | When someone gets a routing exception, the routes are rendered (starting in Rails 4.0). This PR brings parity between the html routes in the `rails/info/routes` path and when rendered from an exception. This is the continuation of #8521 which brought html formatted routes. In addition to bringing parity to the two views, we're keeping our views DRY by rendering off of the same partials. In this case Railties depends on partials provided by ActionDispatch. I'm open to alternative implementations. Ideally both views will use the same code so any improvements or updates to it will be reproduced on both. <hr /> ![](http://f.cl.ly/items/3O1D0K1v0j0i343O3T3T/Screen%20Shot%202012-12-17%20at%203.07.20%20PM.png) | ||||
* | DebugExceptions `File.join` => `File.expand_path` | schneems | 2012-12-16 | 1 | -1/+1 |
| | |||||
* | Remove nodoc from FlashHash #[]= [ci skip] | Pablo Torres | 2012-12-01 | 1 | -1/+1 |
| | |||||
* | Revert "Remove trailing whitespaces" | Florent Guilleux | 2012-12-01 | 1 | -1/+1 |
| | | | | This reverts commit 90c887fa7d0c454b7533e208daefc342dea4d5f3. | ||||
* | Remove trailing whitespaces | Florent Guilleux | 2012-12-01 | 1 | -1/+1 |
| | |||||
* | Merge pull request #8183 from jcoglan/objectless_sessions | Jon Leighton | 2012-11-21 | 1 | -10/+24 |
|\ | | | | | Store FlashHashes in the session as plain hashes | ||||
| * | Store FlashHashes in the session as plain hashes rather than custom objects ↵ | James Coglan | 2012-11-09 | 1 | -10/+24 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | with unstable class names and instance variables. Refactor FlashHash to take values for its ivars in the constructor, to pretty up FlashHash.from_session_value. Remove stale comment on FlashHash: it is no longer Marshaled in the session so we can change its implementation. Remove blank lines I introduced in controller/test_case.rb. Unit tests for FlashHash#to_session_value. Put in a compatibility layer to accept FlashHash serializations from Rails 3.0+. Test that Rails 3.2 session flashes are correctly converted to the new format. Remove code path for processing Rails 3.0 FlashHashes since they can no longer deserialize. Fix session['flash'] deletion condition: it will never be empty?, it will either be nil or a hash with 'discard' and 'flashes' keys. |