request context should not be needed when calling emit() with namespace

[ntamas]

This pull request fixes a minor issue with emit(), which used to require a request context even when the namespace argument was specified. This happens because kwargs.get() evaluates its second argument in an eager manner. The pull request fixes this by replacing kwargs.get() with a membership check, falling back to the namespace of the current request context only if the namespace keyword argument is not present.

[ntamas]

Sorry for being pushy, but is there any chance to merge this into the next release at some point? Currently I'm relying on this patch to make some of our codebase work, but I can't convince Heroku to install Flask-SocketIO from our patched branch on Github instead of PyPI so I can't deploy this fix to Heroku. It would be great if this could be merged eventually.

If you cannot merge this in the near future, let me know and I'll try to work around this in our own codebase instead.

[miguelgrinberg]

Nice and simple, thanks!

I was planning to work on a couple issues before doing another release, but if this is really urgent, I can push one out now and do another in a few days. Let me know.

[ntamas]

Great, and no, it's not that urgent, I can definitely wait for a few more days. Thanks a lot!