Fix SIP options scanning
Pass protocol down to parse_reply for report_* purposes
Disable rubocop RedundantBegin for modules
Split SIP response parsing out on its own, add unit tests.

Passes rspec but fails in framework. WIP.
Refactor SIP response parsing for future improvements
Correct style and doc issues, tidy failure message when not SIP
Move SIP module stuff to Msf::Exploit::Remote::SIP
Use a correct default Accept header -- responses++
Gut SIP UDP stuff, use Msf::Auxiliary::UDPScanner
Also pull the Allow headers (previous behavior)
Enforce proper protocol case where necessary
Show port and protocol when printing service notes, not just name
Address @hmoore-r7's feedback regarding binary encoding
Enforce binary encoding on non-modules, no encoding on modules
Remove unused BATCHSIZE from SIP options_tcp, duplicate from options
Correct SIP header note storage to align with Recog
Clarify SIP mixin method name, store header values as string, etc
Land #3545, fix up sip scanners, msftidy, db services cmd
Merge master back in before landing #3545