{"action":"closed","issue":{"url":"https://api.github.com/repos/crossbario/crossbar/issues/616","repository_url":"https://api.github.com/repos/crossbario/crossbar","labels_url":"https://api.github.com/repos/crossbario/crossbar/issues/616/labels{/name}","comments_url":"https://api.github.com/repos/crossbario/crossbar/issues/616/comments","events_url":"https://api.github.com/repos/crossbario/crossbar/issues/616/events","html_url":"https://github.com/crossbario/crossbar/issues/616","id":129765602,"number":616,"title":"Don't log deliberate exceptions from user/service code","user":{"login":"oberstet","id":233340,"avatar_url":"https://avatars.githubusercontent.com/u/233340?v=3","gravatar_id":"","url":"https://api.github.com/users/oberstet","html_url":"https://github.com/oberstet","followers_url":"https://api.github.com/users/oberstet/followers","following_url":"https://api.github.com/users/oberstet/following{/other_user}","gists_url":"https://api.github.com/users/oberstet/gists{/gist_id}","starred_url":"https://api.github.com/users/oberstet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oberstet/subscriptions","organizations_url":"https://api.github.com/users/oberstet/orgs","repos_url":"https://api.github.com/users/oberstet/repos","events_url":"https://api.github.com/users/oberstet/events{/privacy}","received_events_url":"https://api.github.com/users/oberstet/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/crossbario/crossbar/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/crossbario/crossbar/labels/core","name":"core","color":"009800"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-01-29T13:25:55Z","updated_at":"2016-02-10T20:33:20Z","closed_at":"2016-02-10T20:33:20Z","body":"Raising exceptions from service sessions (`crossbar.router.service.RouterServiceSession`) should NOT be logged. This is similar to running user sessions/components inside a router/container. Such user code explicitly raising should also not log. An exception raised from user code often is used as a valid means to signal call errors.\r\n\r\nE.g. consider https://github.com/crossbario/crossbar/blob/master/crossbar/router/service.py#L531\r\n\r\nThis code may very well raise `wamp.error.history_unavailable` -- and we don't want to clutter the log with:\r\n\r\n```console\r\n2016-01-29T14:20:49+0100 [Controller  12617] Router 'worker1': transport 'transport1' started\r\n2016-01-29T14:20:51+0100 [Router      12621] wamp.error.history_unavailable: \r\n```"}}