kutuluk / loglevel-plugin-remote

A loglevel plugin for sending browser logs to a server
MIT License
102 stars 36 forks source link

Preserve message source when possible #17

Closed aekrylov closed 5 years ago

aekrylov commented 5 years ago

With this change the raw logging function will only be wrapped for messages sent to the server. This will allow for seeing source line in debug instruments (e.g. Chrome Dev tools). For example, we're only sending errors to the server, so seeing where debug and info messages come from is possible. This can lead to confusion though, since not all messages will come from the plugin, so it may seem at first sight that these are plugin's own messages.

kutuluk commented 5 years ago

Мне не нравится эта идея. Поведение плагина становится не консистентным. Этот плагин может быть не последнем в цепочке, за ним вполне может следовать другой плагин, который, например, будет как-то анализировать стектрейс. Так что я считаю, что собственный метод должен вызываться в любом случае, даже если он noop.