search

fourplusone / etherpad-plugins

Plugins for Etherpad Lite
MIT License
23 stars 29 forks source link

several console errors/warnings #24

Closed disy-mk closed 9 years ago

disy-mk commented 11 years ago

EPL on master branch @ v.1.2.8, only ep_headings installed:

after: I was experiencing some notifications of warnings (log4js via mail) when I was on the plugins page; so i shut down the server and restarted (my epl tab closed meanwhile) and opened the admin/plugins url, which triggered the above

after: the scenario from above: going to the troubleshoot page in the admin panel and then back to plugins

after: more/less the same scenario

disy-mk commented 11 years ago

Alright; steps2reproduce & result:

clone latest epl-dev (setup settings.json - admin/plugin pwd..) run bin/run.sh go to admin/pulgins - install ep_headings got to 'troubleshooting' -BANG1 go back to plugins - BANG2

BANG1:

[2013-03-23 17:05:11.846] [ERROR] console - Failed to load 'ep_headings/static/js/index' for 'ep_headings/main/client_hooks/aceEditorCSS': ReferenceError: window is not defined
[2013-03-23 17:05:11.856] [ERROR] console - Failed to load 'ep_headings/static/js/index' for 'ep_headings/main/client_hooks/aceDomLineProcessLineAttributes': ReferenceError: window is not defined
[2013-03-23 17:05:11.861] [ERROR] console - Failed to load 'ep_headings/static/js/index' for 'ep_headings/main/client_hooks/postAceInit': ReferenceError: window is not defined
[2013-03-23 17:05:11.867] [ERROR] console - Failed to load 'ep_headings/static/js/index' for 'ep_headings/main/client_hooks/aceInitialized': ReferenceError: window is not defined
[2013-03-23 17:05:11.873] [ERROR] console - Failed to load 'ep_headings/static/js/index' for 'ep_headings/main/client_hooks/aceAttribsToClasses': ReferenceError: window is not defined
[2013-03-23 17:05:11.878] [ERROR] console - Failed to load 'ep_headings/static/js/index' for 'ep_headings/main/client_hooks/aceRegisterBlockElements': ReferenceError: window is not defined
[2013-03-23 17:05:11.883] [INFO] http - 200, GET /admin/plugins/info
[2013-03-23 17:05:11.883] [INFO] ueberDB - Flushed 1 values

BANG2:

[2013-03-23 17:05:47.258] [INFO] http - 304, GET /admin/plugins
[2013-03-23 17:05:47.259] [INFO] ueberDB - Flushed 1 values
[2013-03-23 17:05:47.469] [INFO] socket.io - handshake authorized 'vefQCMQWuLs0fxvS_QGa'
[2013-03-23 17:05:47.638] [ERROR] console - (node) warning: possible EventEmitter memory leak detected. 11 listeners added. Use emitter.setMaxListeners() to increase limit.
[2013-03-23 17:05:47.642] [ERROR] console - Trace
    at EventEmitter.addListener (events.js:175:15)
    at EventEmitter.withNpm (/home/epl/_epl_129/src/static/js/pluginfw/installer.js:14:9)
    at process.startup.processNextTick.process._tickCallback (node.js:245:9)
fourplusone commented 11 years ago

looks for me like an @ether etherpadl-ite bug. non of the mentioned hooks should be loaded server side as they are declared to be client_hooks.