Prem Sichanugrist (sikachu)

Actually, this ticket should be marked as committed. I forgot >_<

This ticket related to this pull request: https://github.com/rails/rails/pull/258

By the way, I'm backing of on pulling those two Aaron's patch back, as I don't think user would expect that on a stable branch. I don't think I want to break so...

If it's a simple fix, then I'm totally agree with you. Are you going to fix that in Rack::Mount for me, or do you want me to send pull request in?

Anyhow, here'...

@Andrew White: but then the RegExp actually has to be fixed in Rack::Mount, right?

This is a patch for master.

I'm still working on the patch for 3-0-stable, as my patch is currently rely on these two commits: https://github.com/rails/rails/co...

Currently in Rails, when you're adding a wildcard route and run rake routes, you'll usually see an entry like this

routes.rb

match '/*path', :to => 'pages#show...

Thank you for updating my patch and push them for me. ;)

Jeez, seems like I keep overlooking something.

I've update the patch to fixes those issues above, and add those cases as the test.

I've included patch for master and 3-0-stable. The only difference is the CH...

Here's your patch for 3-0-stable, Aaron ;)

I've updated the patch and rebase 002+003 together, per Aaron request