{"action":"created","issue":{"url":"https://api.github.com/repos/SumoLogic/sumologic-collector-docker/issues/48","repository_url":"https://api.github.com/repos/SumoLogic/sumologic-collector-docker","labels_url":"https://api.github.com/repos/SumoLogic/sumologic-collector-docker/issues/48/labels{/name}","comments_url":"https://api.github.com/repos/SumoLogic/sumologic-collector-docker/issues/48/comments","events_url":"https://api.github.com/repos/SumoLogic/sumologic-collector-docker/issues/48/events","html_url":"https://github.com/SumoLogic/sumologic-collector-docker/issues/48","id":246178282,"number":48,"title":"Improper generation of JSON config if escape characters used.","user":{"login":"rwadstein","id":1569069,"avatar_url":"https://avatars2.githubusercontent.com/u/1569069?v=4","gravatar_id":"","url":"https://api.github.com/users/rwadstein","html_url":"https://github.com/rwadstein","followers_url":"https://api.github.com/users/rwadstein/followers","following_url":"https://api.github.com/users/rwadstein/following{/other_user}","gists_url":"https://api.github.com/users/rwadstein/gists{/gist_id}","starred_url":"https://api.github.com/users/rwadstein/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rwadstein/subscriptions","organizations_url":"https://api.github.com/users/rwadstein/orgs","repos_url":"https://api.github.com/users/rwadstein/repos","events_url":"https://api.github.com/users/rwadstein/events{/privacy}","received_events_url":"https://api.github.com/users/rwadstein/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-07-27T21:43:55Z","updated_at":"2017-08-01T21:51:16Z","closed_at":null,"body":"There is a problem with the string substitution when running the `run.sh` and having the `\\` escape character in the config.\r\n\r\nOur regex has a valid JSON format using this for the multiline expression\r\n```\r\n\"manualPrefixRegexp\": \"^\\\\[.*\\\\] \\\\[.*\\\\] \\\\[.*\\\\].*\"\r\n```\r\n\r\nBut when the `run.sh` command is run it strips the extra \\ and outputs this:\r\n```\r\n\"manualPrefixRegexp\": \"^\\[.*\\] \\[.*\\] \\[.*\\].*\"\r\n```\r\n\r\n\r\nIn order for us to get the proper format outputted, we had to add 6 slashes to have it generated properly.\r\n```\r\n\"manualPrefixRegexp\": \"^\\\\\\\\\\\\[.*\\\\\\\\\\\\] \\\\\\\\\\\\[.*\\\\\\\\\\\\] \\\\\\\\\\\\[.*\\\\\\\\\\\\].*\"\r\n```\r\n\r\nIn order to avoid having each container have a crazy escape sequence of characters, we had to have another parser go through all our configs and generate one that conforms to the bug in collector script."},"comment":{"url":"https://api.github.com/repos/SumoLogic/sumologic-collector-docker/issues/comments/319507611","html_url":"https://github.com/SumoLogic/sumologic-collector-docker/issues/48#issuecomment-319507611","issue_url":"https://api.github.com/repos/SumoLogic/sumologic-collector-docker/issues/48","id":319507611,"user":{"login":"maimaisie","id":1904337,"avatar_url":"https://avatars0.githubusercontent.com/u/1904337?v=4","gravatar_id":"","url":"https://api.github.com/users/maimaisie","html_url":"https://github.com/maimaisie","followers_url":"https://api.github.com/users/maimaisie/followers","following_url":"https://api.github.com/users/maimaisie/following{/other_user}","gists_url":"https://api.github.com/users/maimaisie/gists{/gist_id}","starred_url":"https://api.github.com/users/maimaisie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maimaisie/subscriptions","organizations_url":"https://api.github.com/users/maimaisie/orgs","repos_url":"https://api.github.com/users/maimaisie/repos","events_url":"https://api.github.com/users/maimaisie/events{/privacy}","received_events_url":"https://api.github.com/users/maimaisie/received_events","type":"User","site_admin":false},"created_at":"2017-08-01T21:51:16Z","updated_at":"2017-08-01T21:51:16Z","body":"Hi @rwadstein and @kotowick , the fix has been merged and please let us know if you still encounter the same issue with the new images. Thanks."}}
{"actor":{"display_login":"maimaisie"}}