Http print standardization

[jlee-r7]

This implements output standardization for browser exploits via the HttpServer mixin. Format is similar to what was being done manually before but is easier to use (just normal print_status usage) and makes module code easier to read.

Example output:

[*] 192.168.99.1     browser_autopwn - Reporting: {:os_name=>"Linux", :os_lang=>"en-US", :arch=>"x86"}
[*] 192.168.99.1     browser_autopwn - Responding with 2 exploits
[*] 192.168.99.1     java_atomicreferencearray - 192.168.99.1:39437 - Sending Java AtomicReferenceArray Type Violation Vulnerability
[*] 192.168.99.1     java_atomicreferencearray - Generated jar to drop (5284 bytes).

[wchen-r7]

Looking into it right now.

[wchen-r7]

So it looks like if a browser module prints a message like this:
print_status("#{client.peerhost}:#{client.peerport} - Using msvcrt ROP")

The IP:Port will get printed twice. Only one module does this in windows/browsers, and that's fixed in my local branch. Will commit it.

[wchen-r7]

Another scenario that can result the ip:port being printed twice:
print_status("Sending html to #{cli.peerhost}:#{cli.peerport}")

Which will have the following output:
[*] 10.0.1.79 browser_test - Sending html to 10.0.1.79:3995

A lot of modules do this. Here are the ones I found under windows/browser:
http://pastebin.com/raw.php?i=ewgtrUWM

[todb-r7]

I agree with @wchen-r7 -- in order to land this, either the mixin's print_status will have to handle the case where the module's print_status is trying to already print rhost:rport, or all the modules will have to be updated. (It'd be nice to see the latter so future copy-pasters won't screw themselves up)

[wchen-r7]

Talked to egypt. So we've concluded it's probably best to hold on to this pull request until the release.