For some reason, I don't trust @wchen-r7 comment that the rspec failure is
unrelated... :)
On Nov 11, 2013 12:12 PM, "jlee-r7" <notifications@github.com> wrote:

> In lib/msf/core/exploit/browser/server.rb:
>
> > @@ -0,0 +1,510 @@
> > +# -*- coding: binary -*-
> > +
> > +require 'erb'
> > +require 'rex/exploitation/js'
> > +
> > +###
> > +#
> > +# The BrowserExploitServer mixin provides methods to do common tasks seen in modern browser
> > +# exploitation, and is designed to work against common setups such as on Windows, OSX, and Linux.
> > +#
> > +###
> > +
> > +module Msf
> > +  module Exploit::Remote::BrowserExploitServer
>
> Yeah, many old files don't follow the rule. At some point we will go back
> and clean them up, but new files should match their class name. This should
> probably be exploit/remote/browser_exploit_server.rb or maybe
> exploit/remote/http/browser_exploit_server.rb and change the module
> namespace to match, i.e. Exploit::Remote::Http::BrowserExploitServer
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rapid7/metasploit-framework/pull/2613/files#r7567395>
> .
>