search

Studio-42 / elFinder

📁 Open-source file manager for web, written in JavaScript using jQuery and jQuery UI
https://studio-42.github.io/elFinder/
Other
4.6k stars 1.41k forks source link

elFinder.js: Fix multiple calls to elFinder.sync() method when dispat… #3568

Closed magik092 closed 1 year ago

magik092 commented 1 year ago

If the user has browser extensions installed that send a Message then this message is intercepted by js/elFinder.js:4827, and since the message is not a string of type JSON an exception is returned and the self.sync() method is called. Using the example of the wappalyzer add-on - a dozen notifications are sent at once by which elFinder sends multiple requests at once to the application backend.

Here is an example for 32 queries through the wappalyzer add-on: image

nao-pon commented 1 year ago

@magik092 Thank you for your suggestion. It seems better to deal with this problem in elFinder.sync(). I opened a related #3572, so I'm going to close this one.

Thank you again! 👍