I don't believe there is a technical requirement for it. Actually I believe
this was added to make unit testing easier.


On Wed, Mar 26, 2014 at 8:21 PM, jugglinmike <notifications@github.com>wrote:

> In src/client/cloak.js:
>
> >            'force new connection': true
> > -        });
> > +        };
> > +
> > +        if (options['socket.io']) {
> > +          ioOptions = _.extend(ioOptions, options['socket.io']);
>
> Using _.extend here means that users can override the "force new
> connection" option. I'm not familiar enough with the codebase, or the
> design decisions that went into making that a default, so I can't tell if
> that is acceptable or not. If there is a technical requirement for this
> option, or if you feel strongly that it should not be changed (maybe to
> promote consistency across the ecosystem of applications running Cloak, for
> example) you should use _.defaults instead.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/incompl/cloak/pull/74/files#r11006362>
> .
>