Fix broken uses of sock.get that can lead to indefinite hangs

The use of sock.get has no timeout by default, this causes quite a few modules to hang indefinitely if the service they query does not respond. This patch also fixes bad uses of get_once, prefers get_once over get, and corrects some of the check() methods to properly check return values and form correct HTTP requests. 

General cleanup, basically. See this comment for a specific example: https://github.com/hmoore-r7/metasploit-framework/commit/43420aa9844fc4e850b0bfc3f90412801d0a0b79
Here and everywhere else where you are not using a module-specific timeout, why not just call get_once as is, relying on the default values for its parameters (-1, 10)?  On the one hand it is nice to be explicit, but on the other you are now making it such that if someone decides "hey, 20s is a more correct default timeout!" you only have to change it once.  Additionally, a bare get_once is more inline with what was already here.