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