Resolve warning: URI.escape is obsolete. Use Addressable::URI.escape …

…for backwards compatability Signed-off-by: Gavin Reynolds <gavin@chef.io>
arangamani · Sep 23, 2020 · caaabb2 · caaabb2
1 parent 1daaa98
commit caaabb2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/jenkins_api_client.gemspec b/jenkins_api_client.gemspec
@@ -26,5 +26,5 @@ Gem::Specification.new do |s|
   s.add_dependency 'mixlib-shellout', '>= 1.1.0'
   s.add_dependency 'socksify', '>= 1.7.0'
   s.add_dependency 'json', '>= 1.0'
+  s.add_dependency 'addressable', '~> 2.7'
 end
-
diff --git a/lib/jenkins_api_client/urihelper.rb b/lib/jenkins_api_client/urihelper.rb
@@ -1,4 +1,5 @@
 require 'uri'
+require 'addressable/uri'
 
 module JenkinsApi
   module UriHelper
@@ -11,7 +12,7 @@ def form_encode(string)
     # Encode a string for use in the hiearchical part of an URL
     #
     def path_encode(path)
-      URI.escape(path.encode(Encoding::UTF_8))
+      Addressable::URI.escape(path.encode(Encoding::UTF_8))
     end
   end
 end