asvd / jailed

execute untrusted code with custom permissions
MIT License
1.01k stars 69 forks source link

hotfix: add checking 'undefined' in _pluginWeb.js #10

Closed emadurandal closed 9 years ago

emadurandal commented 9 years ago

Hello. Thank you for your kindness earlier in #9 .

After that. I continued to find how to install jailed properly in my meteor project. In server side of meteor, It was easy with /meteorhacks/npm. In client side of meteor, It was not easy.

Finally, I succeed at it with meteor-bower. But, I needed a bit tweak on jailed source code.

So it is this pull request. I'm not sure that this check code is appropriate for your code design. But unless this checking, the switch (m.type) statement cause error at initialization of app.

error_at_initialization

Please consider my proposal.

asvd commented 9 years ago

It looks more like the message is built on the opposite site in some incorrect way. Your proposal would simply hide the error message, but will not solve the problem. Can you reproduce it on clean installation of jailed, or does it only happen when you use it along with meteor?