)]}'
{"octavia/db/repositories.py":[{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"b2daf9991a5e52464db919711e4d24a2b714225b","unresolved":false,"context_lines":[{"line_number":754,"context_line":"                elif (l7policy.action \u003d\u003d"},{"line_number":755,"context_line":"                        constants.L7POLICY_ACTION_REDIRECT_TO_POOL):"},{"line_number":756,"context_line":"                    model_kwargs.update(redirect_url\u003dNone)"},{"line_number":757,"context_line":"            elif l7policy.redirect_url:"},{"line_number":758,"context_line":"                model_kwargs.update("},{"line_number":759,"context_line":"                    action\u003dconstants.L7POLICY_ACTION_REDIRECT_TO_URL)"},{"line_number":760,"context_line":"                model_kwargs.update(redirect_pool_id\u003dNone)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5aef4532_c7e849b7","side":"PARENT","line":757,"range":{"start_line":757,"start_character":38,"end_line":757,"end_character":39},"updated":"2016-03-09 00:28:19.000000000","message":"Also discovered this section was unnecessary after the L7 Policy validation improvements patch: If redirect_url or redirect_pool_id are specified, then the action attribute will be filled in properly by the sanitization step on line 744.","commit_id":"df5f9071851771e78c9f4b0a32c476abe0349678"},{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"b2daf9991a5e52464db919711e4d24a2b714225b","unresolved":false,"context_lines":[{"line_number":772,"context_line":"        # Immediate refresh, as we have found that sqlalchemy will sometimes"},{"line_number":773,"context_line":"        # cache the above query"},{"line_number":774,"context_line":"        session.refresh(listener)"},{"line_number":775,"context_line":"        if position is not None and position \u003c len(listener.l7policies) + 1:"},{"line_number":776,"context_line":"            with session.begin(subtransactions\u003dTrue):"},{"line_number":777,"context_line":"                l7policy_db \u003d listener.l7policies.pop(l7policy_db.position - 1)"},{"line_number":778,"context_line":"                listener.l7policies.insert(position - 1, l7policy_db)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5aef4532_c79a0963","side":"PARENT","line":775,"range":{"start_line":775,"start_character":59,"end_line":775,"end_character":60},"updated":"2016-03-09 00:28:19.000000000","message":"It turns out that after testing this more thoroughly, this section as unnecessary: The squalchemy insert method for Ordering Lists works fine if you \"insert\" beyond the end of the list (the insert just becomes an append).","commit_id":"df5f9071851771e78c9f4b0a32c476abe0349678"}]}