)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"change_message_id":"ae476e869d977f5e982a2e9d88729704c9828158","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added new option sentry_dns that allows Sentry integration"},{"line_number":10,"context_line":"via raven package. The raven log handler is initialized if"},{"line_number":11,"context_line":"the sentry dns_option is non empty string."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I69e9177f7b92a9f610ff49f917c08f39a4b3f1c0"},{"line_number":14,"context_line":"Implements: blueprint sentry-handler"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f485f77_17216dfc","line":11,"range":{"start_line":11,"start_character":4,"end_line":11,"end_character":15},"updated":"2017-11-10 15:14:17.000000000","message":"should be sentry-dsn","commit_id":"682420b2619fa3cd591b0d1ccf5ca8c2c4d226fb"}],"oslo_log/_options.py":[{"author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"change_message_id":"ae476e869d977f5e982a2e9d88729704c9828158","unresolved":false,"context_lines":[{"line_number":104,"context_line":"    cfg.StrOpt(\u0027sentry-dsn\u0027,"},{"line_number":105,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":106,"context_line":"               help\u003d\u0027Enter Sentry DSN if available. \u0027"},{"line_number":107,"context_line":"                    + _IGNORE_MESSAGE),"},{"line_number":108,"context_line":"]"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"generic_log_opts \u003d ["}],"source_content_type":"text/x-python","patch_set":1,"id":"1f485f77_174f2dbe","line":107,"updated":"2017-11-10 15:14:17.000000000","message":"You should add the option secret\u003dTrue. As the URL that you would need to give might contain credentials [1]. Not doing this will log the credentials and potentially leak them.\n\n[1] https://docs.sentry.io/quickstart/#about-the-dsn","commit_id":"682420b2619fa3cd591b0d1ccf5ca8c2c4d226fb"}],"oslo_log/handlers.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"46740444351903a6036c24436fa23a60c224ba88","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"try:"},{"line_number":32,"context_line":"    import raven.handlers.logging"},{"line_number":33,"context_line":"    SentryHandler \u003d raven.handlers.logging.SentryHandler"},{"line_number":34,"context_line":"except ImportError:"},{"line_number":35,"context_line":"    SentryHandler \u003d None"},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"bf659307_c95e6875","line":33,"updated":"2018-04-10 19:33:54.000000000","message":"Is this going to be sufficient? Won\u0027t we need a separate adapter to add extra information available in the context? Or is that not useful?","commit_id":"ba1a2a3ca4de378460e4be9b90471da1e6f6f0de"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"608f1e053291280e3259b9f85df7e0d50526bec9","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"try:"},{"line_number":32,"context_line":"    import raven.handlers.logging"},{"line_number":33,"context_line":"    SentryHandler \u003d raven.handlers.logging.SentryHandler"},{"line_number":34,"context_line":"except ImportError:"},{"line_number":35,"context_line":"    SentryHandler \u003d None"},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"5f7c97a3_e96c8b08","line":33,"in_reply_to":"5f7c97a3_b84d83c6","updated":"2018-07-03 12:57:01.000000000","message":"I don\u0027t know anything about Raven. How does it get OpenStack context information by itself?","commit_id":"ba1a2a3ca4de378460e4be9b90471da1e6f6f0de"},{"author":{"_account_id":26768,"name":"Jan Vondra","email":"jan.vondra@ultimum.io","username":"JanVondra"},"change_message_id":"c5890429a3619145f8567320fd8f26705c5d5c85","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"try:"},{"line_number":32,"context_line":"    import raven.handlers.logging"},{"line_number":33,"context_line":"    SentryHandler \u003d raven.handlers.logging.SentryHandler"},{"line_number":34,"context_line":"except ImportError:"},{"line_number":35,"context_line":"    SentryHandler \u003d None"},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"5f7c97a3_b84d83c6","line":33,"in_reply_to":"bf659307_c95e6875","updated":"2018-07-03 10:17:26.000000000","message":"Raven is capable of getting a context data itself - which means that this is sufficient","commit_id":"ba1a2a3ca4de378460e4be9b90471da1e6f6f0de"}],"setup.cfg":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"46740444351903a6036c24436fa23a60c224ba88","unresolved":false,"context_lines":[{"line_number":27,"context_line":"  fixtures\u003e\u003d3.0.0 # Apache-2.0/BSD"},{"line_number":28,"context_line":"systemd \u003d"},{"line_number":29,"context_line":"  systemd-python\u003e\u003d234 # LGPLv2+"},{"line_number":30,"context_line":"raven \u003d"},{"line_number":31,"context_line":"  raven\u003e\u003d6.0.0 # BSD"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"[entry_points]"}],"source_content_type":"text/x-ttcn-cfg","patch_set":8,"id":"bf659307_294fe4bd","line":30,"updated":"2018-04-10 19:33:54.000000000","message":"Should this be \"sentry\" since that\u0027s the reason for the extra dependency?","commit_id":"ba1a2a3ca4de378460e4be9b90471da1e6f6f0de"}]}