On 08/25/2011 09:40 AM, Jan Provaznik wrote:
On 08/19/2011 09:46 PM, Scott Seago wrote:
src/app/models/provider.rb | 14 ++++++++++++-- src/app/models/provider_account.rb | 16 ++++++++++++++-- src/app/util/condormatic.rb | 2 +- 3 files changed, 27 insertions(+), 5 deletions(-)
diff --git a/src/app/models/provider.rb b/src/app/models/provider.rb index a099c56..c5f64f1 100644 --- a/src/app/models/provider.rb +++ b/src/app/models/provider.rb @@ -64,6 +64,11 @@ class Provider< ActiveRecord::Base
before_destroy :destroyable?
- def encoded_url_with_driver_and_provider
- unencoded = ";driver=#{provider_type.deltacloud_driver}"
- unencoded += ";provider=#{deltacloud_provider}" if
deltacloud_provider
- return url + CGI::escape(unencoded)
Escaped URL doesn't work for me - condor log: 08/25/11 15:37:07 [4589] BaseResource::DoBatchStatus for http://localhost:3002/api%3Bdriver%3Dmock%3Bprovider%3D; skipped for 30 seconds because the resource is down.
W/o escaping (url + unencoded) it works for me.
Ahh, yeah it's escaping the ";driver=" bit too -- I just want to escape deltacloud_provider, I think. I'll resend this patch with that change.
Scott