{"action":"created","issue":{"url":"https://api.github.com/repos/firehol/netdata/issues/3730","repository_url":"https://api.github.com/repos/firehol/netdata","labels_url":"https://api.github.com/repos/firehol/netdata/issues/3730/labels{/name}","comments_url":"https://api.github.com/repos/firehol/netdata/issues/3730/comments","events_url":"https://api.github.com/repos/firehol/netdata/issues/3730/events","html_url":"https://github.com/firehol/netdata/issues/3730","id":324000891,"number":3730,"title":"web_log regexp for squid logs is wrong","user":{"login":"veksh","id":3881519,"avatar_url":"https://avatars3.githubusercontent.com/u/3881519?v=4","gravatar_id":"","url":"https://api.github.com/users/veksh","html_url":"https://github.com/veksh","followers_url":"https://api.github.com/users/veksh/followers","following_url":"https://api.github.com/users/veksh/following{/other_user}","gists_url":"https://api.github.com/users/veksh/gists{/gist_id}","starred_url":"https://api.github.com/users/veksh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/veksh/subscriptions","organizations_url":"https://api.github.com/users/veksh/orgs","repos_url":"https://api.github.com/users/veksh/repos","events_url":"https://api.github.com/users/veksh/events{/privacy}","received_events_url":"https://api.github.com/users/veksh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-05-17T12:27:14Z","updated_at":"2018-05-17T13:55:52Z","closed_at":null,"author_association":"NONE","body":"Range `[\\da-f.:]+` is obviously wrong for matching hostnames, need `[\\da-z.:-]+`; I'll make a merge request shortly\r\n"},"comment":{"url":"https://api.github.com/repos/firehol/netdata/issues/comments/389874964","html_url":"https://github.com/firehol/netdata/issues/3730#issuecomment-389874964","issue_url":"https://api.github.com/repos/firehol/netdata/issues/3730","id":389874964,"user":{"login":"l2isbad","id":22274335,"avatar_url":"https://avatars3.githubusercontent.com/u/22274335?v=4","gravatar_id":"","url":"https://api.github.com/users/l2isbad","html_url":"https://github.com/l2isbad","followers_url":"https://api.github.com/users/l2isbad/followers","following_url":"https://api.github.com/users/l2isbad/following{/other_user}","gists_url":"https://api.github.com/users/l2isbad/gists{/gist_id}","starred_url":"https://api.github.com/users/l2isbad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/l2isbad/subscriptions","organizations_url":"https://api.github.com/users/l2isbad/orgs","repos_url":"https://api.github.com/users/l2isbad/repos","events_url":"https://api.github.com/users/l2isbad/events{/privacy}","received_events_url":"https://api.github.com/users/l2isbad/received_events","type":"User","site_admin":false},"created_at":"2018-05-17T13:55:52Z","updated_at":"2018-05-17T13:55:52Z","author_association":"MEMBER","body":"> maybe extending characters range to include [\\dA-Za-z.:_-]+ will not hurt anyone?\r\n\r\nNo. \r\n\r\nThis is wrong in case of `hostnames`\r\n\r\nhttps://github.com/firehol/netdata/blob/master/python.d/web_log.chart.py#L872-L882\r\n\r\nKeep in mind i am not an expert :smile: \r\n\r\nI think we need custom log format feature for squid_log.\r\n\r\nAnd we can add a list of regexes like in web_log: https://github.com/firehol/netdata/blob/master/python.d/web_log.chart.py#L530-L567\r\nModule will chose the first that matches.\r\n\r\nDo we need requests per hostname chart? What do you think @veksh "}}
{"actor":{"display_login":"l2isbad"}}