)]}'
{"watcher_dashboard/content/audit_templates/forms.py":[{"author":{"_account_id":15949,"name":"Atul Pandey","email":"atul.pandey@nectechnologies.in","username":"atulpandey"},"change_message_id":"3d0bf63cea55e3c51a10ca310c0170c77ac6ae6b","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def __call__(self, value):"},{"line_number":43,"context_line":"        try:"},{"line_number":44,"context_line":"            yaml.safe_load(value)"},{"line_number":45,"context_line":"        except Exception:"},{"line_number":46,"context_line":"            raise core_exc.ValidationError(self.message, code\u003dself.code)"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_b3932c5a","line":44,"range":{"start_line":44,"start_character":17,"end_line":44,"end_character":26},"updated":"2017-01-18 06:26:11.000000000","message":"As per my understanding safe_load doesn\u0027t let to deserialize its own class. So unless it is not required then then safe otherwise we have handle that as well.","commit_id":"eae7f184f5570a46d5401537444d277a6f585e72"},{"author":{"_account_id":18462,"name":"Rahul U Nair","email":"rahulunair@gmail.com","username":"rahulunair"},"change_message_id":"4dc22ab2046c85f88300047d40960a59a77bdbea","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def __call__(self, value):"},{"line_number":43,"context_line":"        try:"},{"line_number":44,"context_line":"            yaml.safe_load(value)"},{"line_number":45,"context_line":"        except Exception:"},{"line_number":46,"context_line":"            raise core_exc.ValidationError(self.message, code\u003dself.code)"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_49c3028b","line":44,"range":{"start_line":44,"start_character":17,"end_line":44,"end_character":26},"in_reply_to":"7a3c09a3_b3932c5a","updated":"2017-01-18 21:16:00.000000000","message":"+1 safe_load will not allow to deserialize user defined classes but in this case as an instance of the YamlValidator is called and the value is passed, it should be safely loaded, not sure if there is any issue with using safe_load here.","commit_id":"eae7f184f5570a46d5401537444d277a6f585e72"}]}
