This project is archived and is in readonly mode.
Id and parameter in path is not properly escaped for ActiveResource
Hi,
we use in project as id sometime also unique strings and it can
contain space. Rails doesn't work in that case and doesn't escape
it.
I attach fix which add escaping also for parameters in path. (patch
is again 2.3.8 if you find it suitable I can do it also for
master)
-
Josef Reidinger- Assigned user set to Rick
-
José Valim- Milestone cleared.
- State changed from new to open
- Assigned user changed from Rick to José Valim
- Importance changed from to Low
Great! Please also do provide a patch for master!
-
Josef Reidinger
- Assigned user changed from José Valim to Rick
Here is patch against master. Also please note that in patch for 2.3 branch is unnecessary changes to require in test file( I again run into issue with shoulda and mocha and it is easy quick workaround - http://blog.anlek.com/2009/10/uninitialized-constant-test-unit-test...)
thanks -
Josef Reidinger
sorry, bad file. this is correct one
-
Josef Reidinger
- Assigned user changed from Rick to José Valim
-
José Valim
- State changed from open to resolved
Applied:
http://github.com/rails/rails/commit/0e969bdaf8ff2e3384350687aa0b58...
Unfortunately I pointed to the wrong ticket in the message. :P
-
