rails/journey — pull request #24 by carlosantoniodasilva (2012-03-09 13:58:26+00:00)
Title: Fix to not allow trailing slash on script name when matching

This is related to a Rails issue with mounted engines on root path.
See Rails issues:

https://github.com/rails/rails/issues/2131
https://github.com/rails/rails/issues/2230
https://github.com/rails/rails/issues/2579
https://github.com/rails/rails/issues/5342

*Related*, [this issue](https://github.com/rails/rails/pull/2577) added a fix for url generation with double slashes in engines, and ended up adding by mistake the same failing test case that 5342 introduces again. The failing test case [was removed later](https://github.com/rails/rails/commit/04baa4b2cad105718ac6acda3d82b6521c4536ea), but the issue related to it still exists.

I believe this could be merged in versions for both Rails master and 3-2. Also, issue 5342 in Rails can be merged in both as well to avoid regressions, I'll add a comment there.

Let me know if something can be improved. Thanks!