Merge pull request #1863 from beefproject/zombie_id_fix

update the destroy to destroy all

core/main/rest/handlers/hookedbrowsers.rb

@@ -41,30 +41,30 @@ module BeEF
           error 401 unless hb != nil
 
           details = BeEF::Core::Models::BrowserDetails.where(:session_id => hb.session)
-	  details.destroy
+	  details.destroy_all
 
 	  logs = BeEF::Core::Models::Log.where(:hooked_browser_id => hb.id)
-	  logs.destroy
+	  logs.destroy_all
 
 	  commands = BeEF::Core::Models::Command.where(:hooked_browser_id => hb.id)
-	  commands.destroy
+	  commands.destroy_all
 
 	  results = BeEF::Core::Models::Result.where(:hooked_browser_id => hb.id)
-	  results.destroy
+	  results.destroy_all
 
 	  begin
 	    requester = BeEF::Core::Models::Http.where(:hooked_browser_id => hb.id)
-	    requester.destroy
+	    requester.destroy_all
 	  rescue => e
 	    #the requester module may not be enabled
 	  end
 
 	  begin
 	    xssraysscans = BeEF::Core::Models::Xssraysscan.where(:hooked_browser_id => hb.id)
-	    xssraysscans.destroy
+	    xssraysscans.destroy_all
 
 	    xssraysdetails = BeEF::Core::Models::Xssraysdetail.where(:hooked_browser_id => hb.id)
-	    xssraysdetails.destroy
+	    xssraysdetails.destroy_all
 	  rescue => e
 	    #the xssraysscan module may not be enabled
 	  end

spec/beef/core/main/handlers/browser_details_handler_spec.rb

@@ -72,5 +72,14 @@ RSpec.describe 'Browser details handler' do
 
       	
 	end
- 
+	
+	it 'can successfully hook a browser' do
+		@token  = BeefRestClient.new('http', ATTACK_DOMAIN, '3000', BEEF_USER, BEEF_PASSWD).auth()[:token]
+		victim = BeefTest.new_victim
+		sleep(3)
+      	response = RestClient.get "#{RESTAPI_HOOKS}", {:params => {:token => @token}}
+		  x = JSON.parse(response.body)
+		  puts x
+		expect(x)
+	end
 end