{"action":"opened","issue":{"url":"https://api.github.com/repos/RequestPolicyContinued/requestpolicy/issues/593","labels_url":"https://api.github.com/repos/RequestPolicyContinued/requestpolicy/issues/593/labels{/name}","comments_url":"https://api.github.com/repos/RequestPolicyContinued/requestpolicy/issues/593/comments","events_url":"https://api.github.com/repos/RequestPolicyContinued/requestpolicy/issues/593/events","html_url":"https://github.com/RequestPolicyContinued/requestpolicy/issues/593","id":56127364,"number":593,"title":"logging: reduce number of logging messages in \"release\" versions","user":{"login":"myrdd","id":4286829,"avatar_url":"https://avatars.githubusercontent.com/u/4286829?v=3","gravatar_id":"","url":"https://api.github.com/users/myrdd","html_url":"https://github.com/myrdd","followers_url":"https://api.github.com/users/myrdd/followers","following_url":"https://api.github.com/users/myrdd/following{/other_user}","gists_url":"https://api.github.com/users/myrdd/gists{/gist_id}","starred_url":"https://api.github.com/users/myrdd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/myrdd/subscriptions","organizations_url":"https://api.github.com/users/myrdd/orgs","repos_url":"https://api.github.com/users/myrdd/repos","events_url":"https://api.github.com/users/myrdd/events{/privacy}","received_events_url":"https://api.github.com/users/myrdd/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RequestPolicyContinued/requestpolicy/labels/optimization","name":"optimization","color":"dddddd"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-01-31T14:54:45Z","updated_at":"2015-01-31T14:54:45Z","closed_at":null,"body":"Currently, logging is mainly used for development. **Most** people do not need logging. — Note: I'm talking about logging to `stdout`, not to the \"Request Log\".\r\n\r\nThe issue is that there might be `Logger` calls in time-critical functions or even in loops. I think the release version should not contain such loggings. **Benefits:**\r\n\r\n- normal usage → better performance\r\n- *development and debugging* → different levels of verbosity could be specified when creating the xpi, without the need to edit code. (Currently some very wordy loggings are simply commented out.)\r\n\r\nThe solution I could think of is using a preprocessor, e.g. https://www.npmjs.com/package/preprocessor.\r\n\r\n"}}