On 08/26/2011 01:58 PM, Scott Seago wrote:
- def encoded_url_with_driver_and_provider
- url_extras = ";driver=#{provider_type.deltacloud_driver}"
- if deltacloud_provider
url_extras += ";provider=#{CGI::escape(deltacloud_provider)}"
- end
- return url + url_extras
- end
One additional comment w/ this patch. If deltacloud_provider is specified, the resulting url will be:
;driver=#{provider_type.deltacloud_driver};provider=#{CGI::escape(deltacloud_provider)}
whereas what I believe we want is
;driver=#{provider_type.deltacloud_driver}&provider=#{CGI::escape(deltacloud_provider)}
Note the 2nd ';' is now an '&' to conform to http guidelines.
-Mo